Skip to content

Comments

Add cards test app#150

Merged
heyitsaamir merged 6 commits intomainfrom
aamirj/addCardsTest
Sep 12, 2025
Merged

Add cards test app#150
heyitsaamir merged 6 commits intomainfrom
aamirj/addCardsTest

Conversation

@heyitsaamir
Copy link
Collaborator

@heyitsaamir heyitsaamir commented Sep 12, 2025

  • Tests the cards package out. Found some bugs in serialization
  • Adds a cards test
  • Made sure these things work from the snippets that are in the docs

PR Dependency Tree

This tree was auto-generated by Charcoal

Copilot AI review requested due to automatic review settings September 12, 2025 00:07
Copy link
Contributor

Copilot AI left a comment

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 a new cards test application to validate the adaptive cards functionality within the Microsoft Teams AI Library. The test demonstrates basic message handling using the Teams Apps framework and validates card serialization features that were found to have bugs during development.

  • Adds a complete cards test application with proper project structure
  • Updates logging level for graph token failures from warning to debug
  • Includes a template manifest file for test applications

Reviewed Changes

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

Show a summary per file
File Description
tests/cards/src/main.py Creates main test application with message handling to validate cards functionality
tests/cards/pyproject.toml Project configuration for the cards test with required dependencies
tests/cards/README.md Documentation template for the cards test application
templates/test/{{cookiecutter.test_app_name}}/appPackage/manifest.json Teams app manifest template for test applications
packages/apps/src/microsoft/teams/apps/app.py Changes graph token failure logging from warning to debug level

@heyitsaamir heyitsaamir changed the title Add cards test Add cards test app Sep 12, 2025
@heyitsaamir heyitsaamir merged commit bdd327a into main Sep 12, 2025
6 checks passed
@heyitsaamir heyitsaamir deleted the aamirj/addCardsTest branch September 12, 2025 17:01
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