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

Add tests to ensure database connections work inside the container #1

Merged
merged 43 commits into from
Sep 9, 2019
Merged

Add tests to ensure database connections work inside the container #1

merged 43 commits into from
Sep 9, 2019

Conversation

kabirbaidhya
Copy link
Contributor

@kabirbaidhya kabirbaidhya commented Sep 6, 2019 •
edited
Loading

Add tests to ensure database connections work inside the container.

  • Configure Travis CI
  • Use multi-stage builds
  • Check docker builds on the CI
  • Test Cases:
    • Test connection to MS SQL Server
    • Test connection to Postgresql Server

@kabirbaidhya kabirbaidhya changed the title (WIP) Add tests for database connections established using pyodbc (WIP) Add tests to ensure database connections work inside the container Sep 6, 2019
@kabirbaidhya kabirbaidhya force-pushed the test-pyodbc branch 2 times, most recently from 3e8de91 to 21f7c2b Compare September 6, 2019 17:12
@kabirbaidhya kabirbaidhya force-pushed the test-pyodbc branch 2 times, most recently from 42ed7da to b4e3872 Compare September 6, 2019 18:21
@kabirbaidhya kabirbaidhya added the pg label Sep 6, 2019
@kabirbaidhya kabirbaidhya changed the title (WIP) Add tests to ensure database connections work inside the container Add tests to ensure database connections work inside the container Sep 6, 2019
@kabirbaidhya kabirbaidhya marked this pull request as ready for review September 6, 2019 18:44
@SafalPandey SafalPandey merged commit 71b68ff into master Sep 9, 2019
@SafalPandey SafalPandey deleted the test-pyodbc branch September 9, 2019 08:40
kabirbaidhya pushed a commit that referenced this pull request Oct 27, 2019
@shradayshakya
Merge pull request #1 from laudio/master
8a32241 
Update
@kabirbaidhya kabirbaidhya mentioned this pull request Nov 26, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SafalPandey SafalPandey SafalPandey approved these changes

@cham11ng cham11ng Awaiting requested review from cham11ng

@mesaugat mesaugat Awaiting requested review from mesaugat

Assignees
No one assigned
Labels
mssql pg test
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kabirbaidhya @SafalPandey