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.

Sign up for GitHub

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

Jump to bottom

Fix integration tests #496

Merged
merged 4 commits into from
Apr 28, 2024
Merged

Fix integration tests #496

merged 4 commits into from
Apr 28, 2024

Conversation

erikbra
Copy link
Owner

@erikbra erikbra commented Apr 11, 2024 •
edited

  • Maxparallel 1
  • Less tests - takes less time, while debugging
  • Running the test DLLs directly to better discover errors
  • Try running with multiple postgres'es
  • Added test report step
  • Try turning off parallelization
  • Fixed a couple of errors
  • Enable all DBs, change name of test report
  • More verbose debug message when trying to connect to non-existing db
  • Set timeout on the job
  • Working on Oracle
  • Added Drop of all test DBs after running test
  • Timeout 12 mins
  • Separate test steps in CmdLine and DLLs
  • normalize line endings in error message before checking
  • Some Oracle fixes
  • Run 'build failure' steps on not success (not working)
  • Smink v 0.4.0
  • Disable Azure tests for now (too brittle, and too expensive)
  • Add Docker based tests for all DBs (only form ubuntu client, though. But, better than nothing)

@erikbra erikbra temporarily deployed to integration-tests April 11, 2024 21:20 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 11, 2024 21:30 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 11, 2024 21:30 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 11, 2024 21:30 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 25, 2024 05:55 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 25, 2024 05:55 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 25, 2024 05:55 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 25, 2024 05:55 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 25, 2024 05:55 — with GitHub Actions Inactive
@erikbra erikbra temporarily deployed to integration-tests April 25, 2024 07:08 — with GitHub Actions Inactive
@erikbra erikbra force-pushed the chore/fix-integration-tests branch from 30b60e2 to b00d916 Compare April 25, 2024 14:46
@erikbra
Trying to fix the integration tests
60de0da 
* Maxparallel 1
* Less tests - takes less time, while debugging
* Running the test DLLs directly to better discover errors
* Try running with multiple postgres'es
* Added test report step
* Try turning off parallelization
* Fixed a couple of errors
* Enable all DBs, change name of test report
* More verbose debug message when trying to connect to non-existing db
* Set timeout on the job
* Working on Oracle
* Added Drop of all test DBs after running test
* Timeout 12 mins
* Separate test steps in CmdLine and DLLs
* normalize line endings in error message before checking
* Some Oracle fixes
* Run 'build failure' steps on not success (not working)
* Smink v 0.4.0
* Disable Azure tests for now (too brittle, and too expensive)
@erikbra erikbra force-pushed the chore/fix-integration-tests branch from b00d916 to 60de0da Compare April 25, 2024 14:54
@erikbra erikbra marked this pull request as ready for review April 25, 2024 14:55
@erikbra erikbra merged commit 296a062 into main Apr 28, 2024
41 checks passed
@erikbra erikbra deleted the chore/fix-integration-tests branch April 28, 2024 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@erikbra