Skip to content

Conversation

@mathurf
Copy link
Contributor

@mathurf mathurf commented Oct 10, 2019

  • Fixed failed tests caused by the auto-crlf on git on Windows
    Fixing it by adding a .gitattributes file that force .pickle file to have Unix eols (LF)

- Fixed failed tests caused by the auto-crlf on git on Windows
- Fixed 'sudo_as_login' fails test:
It looks like the server is returning a different error than the one the test expects.  The error the server is returning seems correct for what's happening, So Will thinks that something changed SG-side and we need to update the test to match.
It's looking for an error that says The user does not have permission to 'sudo':
Instead it's seeing Cannot 'sudo' - unknown or retired user:
Which, given that blah isn't a real user, makes sense
@mathurf mathurf requested a review from willis102 October 10, 2019 20:11
@coveralls
Copy link

coveralls commented Oct 10, 2019

Pull Request Test Coverage Report for Build 1266

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 68.723%

Totals Coverage Status
Change from base Build 1259: 0.0%
Covered Lines: 6752
Relevant Lines: 9825

💛 - Coveralls

@coveralls
Copy link

coveralls commented Oct 10, 2019

Pull Request Test Coverage Report for Build 1263

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 68.723%

Totals Coverage Status
Change from base Build 1259: 0.0%
Covered Lines: 6752
Relevant Lines: 9825

💛 - Coveralls

Copy link
Contributor

@willis102 willis102 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mathurf mathurf merged commit d0d134d into master Oct 11, 2019
@mathurf mathurf deleted the ticket/SG-13480_fix_tests branch October 11, 2019 14:45
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.

4 participants