Skip to content

Add GitHub Copilot environment setup and increase test coverage to 87… - #5

Merged
marcelo-m7 merged 1 commit into
copilot/create-api-for-course-catalogfrom
fix-enviroment
Oct 27, 2025
Merged

marcelo-m7 merged 1 commit into
copilot/create-api-for-course-catalogfrom
fix-enviroment

Conversation

@marcelo-m7

Copy link
Copy Markdown
Contributor

…% (#4)

  • Checkpoint from VS Code for coding agent session

  • Add Copilot setup workflow and improve test coverage to 87%

  • Address code review feedback - improve test assertions


Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

  • Test A
  • Test B

Test Configuration:

  • Python version:
  • Operating System:

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

#4)

* Checkpoint from VS Code for coding agent session

* Add Copilot setup workflow and improve test coverage to 87%

Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>

* Address code review feedback - improve test assertions

Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>

---------

Co-authored-by: marcelo-m7 <marcelo.net77@hotmail.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>
@marcelo-m7
marcelo-m7 requested a review from Copilot October 27, 2025 11:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds GitHub Copilot environment configuration and significantly improves test coverage to 87% by adding comprehensive test cases for edge cases and error handling scenarios.

Key changes:

  • Added Copilot setup workflow (.github/workflows/copilot-setup-steps.yml) and instructions (.github/copilot-instructions.md) to configure the development environment for GitHub Copilot agents
  • Enhanced test coverage in tests/test_scrape_ualg.py with 7 new test cases covering document saving, error handling, course page parsing, and scraping limits
  • Added 4 new test cases in tests/test_api.py for filtering, pagination, and fallback behavior

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/copilot-setup-steps.yml New GitHub Actions workflow that sets up Python environment, installs dependencies, and initializes database for Copilot agents
.github/copilot-instructions.md Comprehensive documentation for Copilot agents describing architecture, workflows, conventions, and extension patterns
tests/test_scrape_ualg.py Added 7 new test methods covering document operations, module/area parsing, scraping limits, and error handling
tests/test_api.py Added 4 new test methods for API filtering by school/area, pagination with offset, and root endpoint fallback

@marcelo-m7
marcelo-m7 merged commit 824aa91 into copilot/create-api-for-course-catalog Oct 27, 2025
2 checks passed
@marcelo-m7
marcelo-m7 deleted the fix-enviroment branch October 27, 2025 11:19
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.

3 participants