Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add unit tests for SsisDeployment #22

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

GooseLF
Copy link
Contributor

@GooseLF GooseLF commented Jun 23, 2022

  • Added unit tests for SssiDeployment.

@github-actions
Copy link

github-actions bot commented Jun 23, 2022

Unit Test Results

41 tests  +41   40 ✔️ +40   0s ⏱️ ±0s
  1 suites +  1     1 💤 +  1 
  1 files   +  1     0 ±  0 

Results for commit d1a6922. ± Comparison against base commit 020903c.

♻️ This comment has been updated with latest results.

@gvee-uk
Copy link
Contributor

gvee-uk commented Jun 24, 2022

Unit Test Results

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s 0 suites ±0   0 💤 ±0  0 files   ±0   0 ±0 

Results for commit 56d9a92. ± Comparison against base commit 020903c.

♻️ This comment has been updated with latest results.

@GooseLF something's not right here - it's reporting zero tests 🤔

src/model.py Outdated Show resolved Hide resolved
src/model.py Outdated Show resolved Hide resolved
test/conftest.py Outdated Show resolved Hide resolved
test/conftest.py Outdated Show resolved Hide resolved
@GooseLF GooseLF force-pushed the issue/17_SsisDeployment_Tests branch from 56d9a92 to 3da57ae Compare June 24, 2022 09:15
@GooseLF GooseLF requested a review from gvee-uk June 24, 2022 09:16
test/test_model.py Outdated Show resolved Hide resolved
@GooseLF GooseLF force-pushed the issue/17_SsisDeployment_Tests branch from 3da57ae to d1a6922 Compare June 24, 2022 09:43
@GooseLF GooseLF requested a review from gvee-uk June 24, 2022 09:44
@GooseLF GooseLF merged commit 60cae75 into main Jun 24, 2022
@GooseLF GooseLF deleted the issue/17_SsisDeployment_Tests branch June 24, 2022 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants