8000 Add e2e test + improve logging by nimrossum · Pull Request #633 · git-truck/git-truck · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add e2e test + improve logging #633

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

Merged
merged 12 commits into from
May 20, 2023
Merged

Add e2e test + improve logging #633

merged 12 commits into from
May 20, 2023

Conversation

nimrossum
Copy link
Contributor
@nimrossum nimrossum commented May 18, 2023

Fixes #345
Fixes #87

@codecov
Copy link
codecov bot commented May 18, 2023

Codecov Report

Merging #633 (102ef6c) into main (dec0afc) will not change coverage.
The diff coverage is n/a.

❗ Current head 102ef6c differs from pull request most recent head 2a61b6a. Consider uploading reports for the commit 2a61b6a to get more accurate results

@@           Coverage Diff           @@
##             main     #633   +/-   ##
=======================================
  Coverage   59.52%   59.52%           
=======================================
  Files           2        2           
  Lines          42       42           
  Branches        7        7           
=======================================
  Hits           25       25           
  Misses         16       16           
  Partials        1        1           

@nimrossum nimrossum changed the title Small changes Add e2e test + improve logging May 19, 2023
@nimrossum
Copy link
Contributor Author

E2E failing due to the checked out branch being the PR branch, but the test attempts running against main, which is not checked out in the workflow.

  • Either change the target to be the current branch or make sure the main branch is checked out. Also remember to use check out depth, to include the history.

@nimrossum nimrossum requested review from emiljapelt and tjomson and removed request for emiljapelt May 20, 2023 15:55
@nimrossum nimrossum merged commit a18eb99 into main May 20, 2023
@nimrossum nimrossum deleted the logging branch May 20, 2023 17:11
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.

Color log prefix Setup Integration test
1 participant
0