Feature: Retrive site information trough webservice

[erseco]

This pull request adds a new "site" command group to the CLI, enabling users to fetch and display detailed Moodle site information. It introduces new data models for site details, extends the session API for generic webservice calls, and provides test coverage for the new functionality.

New CLI feature: Site info commands

• Added a new site command group to the CLI, allowing users to retrieve site information via py-moodle site info, with output available in both table and JSON formats (src/py_moodle/cli/app.py, src/py_moodle/cli/site.py). [1] [2] [3]

Data modeling and API integration

• Introduced new dataclasses (SiteInfo, SiteFunction, AdvancedFeature) to model the site information returned by the Moodle API, and implemented the get_site_info function to fetch and parse this data (src/py_moodle/site.py).

Session API improvements

• Added a generic call method to MoodleSession for making arbitrary Moodle webservice API calls, which is now used to implement site info retrieval (src/py_moodle/session.py). [1] [2]

Testing

• Added a test to verify that get_site_info correctly retrieves and parses site information from a real Moodle environment (tests/test_site.py).

[Copilot]

Pull Request Overview

This pull request introduces a new CLI command group for retrieving Moodle site information through webservice APIs. The feature allows users to fetch comprehensive site details and display them in either table or JSON format.

• Adds a new site command group with info subcommand to the CLI
• Introduces structured data models for site information, functions, and advanced features
• Extends the session API with a generic call method for arbitrary webservice requests

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/py_moodle/site.py	Defines data models (SiteInfo, SiteFunction, AdvancedFeature) and implements get_site_info function
src/py_moodle/session.py	Adds generic call method to MoodleSession for webservice API calls
src/py_moodle/cli/site.py	Implements site CLI command group with info subcommand and JSON/table output options
src/py_moodle/cli/app.py	Registers the new site command group with the main CLI application
tests/test_site.py	Adds integration test for get_site_info function

Comments suppressed due to low confidence (1)

tests/test_site.py:7

• [nitpick] The function name 'test_get_site_info_real' could be clearer. Consider renaming to 'test_get_site_info_integration' or 'test_get_site_info_with_live_session' to better indicate this is an integration test.

def test_get_site_info_real(request):

[Copilot]

The DataclassEncoder class is defined inside the function scope. Consider moving this to module level or a utility module to improve reusability and avoid recreating the class on each function call.

Suggested change

	class DataclassEncoder(json.JSONEncoder):
	def default(self, o):
	from dataclasses import asdict, is_dataclass
	if is_dataclass(o):
	return asdict(o)
	return super().default(o)