Skip to content

Commit

Permalink
CI: Don’t skip test_run_with_user_global_config_file
Browse files Browse the repository at this point in the history
Before this change one of yamllint’s tests wasn’t being run on GitHub
Actions Runners because the HOME environment variable couldn’t be
overridden. I just tested it, and it looks like the HOME variable can be
overridden now, so that test no longer needs to be skipped.
  • Loading branch information
Jayman2000 authored and adrienverge committed Jan 4, 2024
1 parent 92688d2 commit 3513ec1
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,6 @@ def test_run_with_config_file(self):
cli.run(('-c', f.name, os.path.join(self.wd, 'a.yaml')))
self.assertEqual(ctx.returncode, 1)

@unittest.skipIf(os.environ.get('GITHUB_RUN_ID'), '$HOME not overridable')
def test_run_with_user_global_config_file(self):
home = os.path.join(self.wd, 'fake-home')
dir = os.path.join(home, '.config', 'yamllint')
Expand Down

0 comments on commit 3513ec1

Please sign in to comment.