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

changes to test on windows #999

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

daniel-edwards-iress
Copy link

🔧 Changes

  • Changed the way the CLI determines the config directory to use. Uses os.UserConfigDir
  • Added a config file override environment variable AUTH0_CONFIG_FILE
  • Created a script to run unit tests on Windows
  • Added a job to GitHub actions to run unit tests on Windows

🔬 Testing

I have added unit tests and a test runner for Windows to check unit tests pass.

It should be passing the CI testing, and by running it on Windows the config will get stored in a user's config directory without a user needing to manually set a HOME environment variable.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@daniel-edwards-iress daniel-edwards-iress requested a review from a team as a code owner May 9, 2024 00:39
@daniel-edwards-iress daniel-edwards-iress requested a review from a team as a code owner August 15, 2024 01:42
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.

1 participant