Skip to content

Conversation

Copy link

Copilot AI commented Jan 21, 2026

Windows CI tests were failing at Git checkout due to invalid filenames containing colons. Additionally, tests referenced non-existent data directories and used an undefined environment variable for venv activation.

Changes

  • Filename compatibility: Renamed 14 CSV test files replacing : with - in timestamps

    • Windows filesystems reject colons in paths
    • Changed format: 2022-12-23 16:00:002022-12-23 16-00-00
    • Affected files in tests/resources/data/ and tests/resources/market_data_sources_for_testing/
  • Test matrix: Added windows-latest to OS matrix in .github/workflows/test.yml

  • venv activation: Fixed platform-specific paths

    if [ "$RUNNER_OS" == "Windows" ]; then
      source .venv/Scripts/activate
    else
      source .venv/bin/activate
    fi

    Previously used undefined $VENV variable

  • Test data: Created tests/resources/backtest_reports_for_testing/

    • Added checkpoints.json with 16 algorithm IDs across 2 date ranges
    • Created 16 algorithm directories with algorithm_id.json files
    • Required by TestGetCheckpoints test suite

The filename issue was the primary blocker preventing repository checkout on Windows runners.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/coding-kitties/investing-algorithm-framework/actions/runs/21254732255/jobs
    • Triggering command: /usr/bin/curl curl -s -H Accept: application/vnd.github.v3+json REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix failing tests to run on Windows VMs Fix test failures on Windows by adding OS support and missing test data Jan 21, 2026
Copilot AI requested a review from MDUYN January 21, 2026 16:49
Copilot AI changed the title Fix test failures on Windows by adding OS support and missing test data Fix Windows test failures: filename compatibility and missing test data Jan 22, 2026
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.

2 participants