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

update docs for go server tests #1812

Merged
merged 1 commit into from Mar 4, 2019

Conversation

2 participants
@lynzt
Copy link
Contributor

lynzt commented Mar 4, 2019

Description

Due to this pr: #1770, we updated the test_db port number to 5433. Update the docs to explain how to run 1 off server tests by passing in the port number.

Setup

make db_test_e2e_populate
Run any individual server test described here: https://github.com/transcom/mymove/blob/master/docs/how-to/run-go-tests.md

Screenshots

The error: pq: database "test_db" does not exist
screen shot 2019-03-04 at 3 23 59 pm

@codecov

This comment has been minimized.

Copy link

codecov bot commented Mar 4, 2019

Codecov Report

Merging #1812 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1812      +/-   ##
==========================================
+ Coverage   49.11%   49.15%   +0.03%     
==========================================
  Files         425      425              
  Lines       18174    18136      -38     
  Branches     1630     1625       -5     
==========================================
- Hits         8926     8914      -12     
+ Misses       8468     8442      -26     
  Partials      780      780

@lynzt lynzt requested review from jim , chrisgilmerproj and rdhariwal Mar 4, 2019

@chrisgilmerproj
Copy link
Contributor

chrisgilmerproj left a comment

🚀 - Thanks!

@lynzt lynzt merged commit 1a3ca6f into master Mar 4, 2019

18 of 19 checks passed

Codacy/PR Quality Review Hang in there, Codacy is reviewing your Pull request.
Details
ci/circleci: acceptance_tests_experimental Your tests passed on CircleCI!
Details
ci/circleci: acceptance_tests_local Your tests passed on CircleCI!
Details
ci/circleci: acceptance_tests_staging Your tests passed on CircleCI!
Details
ci/circleci: build_app Your tests passed on CircleCI!
Details
ci/circleci: build_migrations Your tests passed on CircleCI!
Details
ci/circleci: build_tools Your tests passed on CircleCI!
Details
ci/circleci: client_test Your tests passed on CircleCI!
Details
ci/circleci: client_test_coverage Your tests passed on CircleCI!
Details
ci/circleci: integration_tests_mymove Your tests passed on CircleCI!
Details
ci/circleci: integration_tests_office Your tests passed on CircleCI!
Details
ci/circleci: integration_tests_tsp Your tests passed on CircleCI!
Details
ci/circleci: pre_deps_golang Your tests passed on CircleCI!
Details
ci/circleci: pre_deps_yarn Your tests passed on CircleCI!
Details
ci/circleci: pre_test Your tests passed on CircleCI!
Details
ci/circleci: server_test Your tests passed on CircleCI!
Details
ci/circleci: server_test_coverage Your tests passed on CircleCI!
Details
codecov/patch Coverage not affected when comparing 586cb3f...40a77c1
Details
codecov/project 49.15% (+0.03%) compared to 586cb3f
Details

@lynzt lynzt deleted the lt-rd-update-go-test-documentation branch Mar 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.