Remove secrets from integration tests

laurennat commented 2 weeks ago

Removed secret use from integration tests, and instead, make local docker containers to run the tests

coveralls commented 2 weeks ago

Pull Request Test Coverage Report for Build 9456106644

Details

0 of 0 changed or added relevant lines in 0 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.008%) to 41.733%

Files with Coverage Reduction	New Missed Lines	%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
extensions/sql-database-projects/src/common/httpClient.ts	1	66.18%
extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts	2	52.03%
<!--	Total:	4	-->

Totals
Change from base Build 9371615754:	-0.008%
Covered Lines:	30718
Relevant Lines:	68884

Pull Request Test Coverage Report for Build 9456122270

Details

0 of 0 changed or added relevant lines in 0 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.004%) to 41.737%

Files with Coverage Reduction	New Missed Lines	%
src/sql/workbench/services/connection/browser/connectionDialogWidget.ts	1	73.44%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
extensions/notebook/src/dialog/configurePython/configurePythonWizard.ts	1	45.22%
<!--	Total:	3	-->

Totals
Change from base Build 9371615754:	-0.004%
Covered Lines:	30719
Relevant Lines:	68884

💛 - Coveralls

Benjin commented 2 weeks ago

Can you update extensions/integration-tests/readme.md with the new instructions for how to run these integration tests?

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9553005193

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.008%) to 41.733%

Files with Coverage Reduction	New Missed Lines	%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
extensions/sql-database-projects/src/common/httpClient.ts	1	66.18%
extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts	2	52.03%
<!--	Total:	4	-->

Totals
Change from base Build 9371615754:	-0.008%
Covered Lines:	30718
Relevant Lines:	68884

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9555351759

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 41.739%

Files with Coverage Reduction	New Missed Lines	%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
<!--	Total:	1		-->

Totals
Change from base Build 9371615754:	-0.002%
Covered Lines:	30721
Relevant Lines:	68884

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9556629344

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.003%) to 41.738%

Files with Coverage Reduction	New Missed Lines	%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
extensions/notebook/src/dialog/configurePython/configurePythonWizard.ts	1	45.22%
<!--	Total:	2	-->

Totals
Change from base Build 9371615754:	-0.003%
Covered Lines:	30720
Relevant Lines:	68884

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9570342529

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.003%) to 41.738%

Files with Coverage Reduction	New Missed Lines	%
src/sql/workbench/services/connection/browser/connectionDialogWidget.ts	1	73.44%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
<!--	Total:	2	-->

Totals
Change from base Build 9371615754:	-0.003%
Covered Lines:	30720
Relevant Lines:	68884

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9570649693

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.003%) to 41.738%

Files with Coverage Reduction	New Missed Lines	%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
extensions/notebook/src/dialog/configurePython/configurePythonWizard.ts	1	45.22%
<!--	Total:	2	-->

Totals
Change from base Build 9371615754:	-0.003%
Covered Lines:	30720
Relevant Lines:	68884

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9570927116

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.002%) to 41.743%

Files with Coverage Reduction	New Missed Lines	%
src/sql/workbench/services/connection/browser/connectionDialogWidget.ts	1	73.44%
extensions/notebook/src/book/bookTreeView.ts	1	36.91%
<!--	Total:	2	-->

Totals
Change from base Build 9371615754:	0.002%
Covered Lines:	30723
Relevant Lines:	68884

microsoft / azuredatastudio