search

xmtp / example-notification-server-go

Example push notification server, written in Golang
MIT License
15 stars 8 forks source link

Add integration test suite #32

Closed neekolas closed 5 months ago

neekolas commented 6 months ago

Summary

Generated summary (powered by Graphite) > ## TL;DR > This pull request includes changes to multiple files related to integration testing, environment variables, Docker configuration, and code modifications. It adds new services, updates job names, and includes new configuration settings for an integration service. > > ## What changed > - Updated .dockerignore to ignore specific directories and files > - Added environment variables to .env.docker for configuration > - Modified test.yml in .github/workflows to update job and step names > - Renamed a job from "Test" to "Unit" > - Added a new job for integration tests > - Modified .gitignore to include "node_modules" > - Added a condition for enabling HTTP delivery in main.go > - Added a new HTTP delivery service in main.go > - Created new integration scripts in the dev folder > - Modified up script in the dev folder for specific service setup > - Added new Dockerfile to the integration directory > - Added a README.md file in the integration directory > - Updated package.json with dependencies and devDependencies > - Added new TypeScript files for configuration and message handling > > ## How to test > 1. Check the updated .dockerignore, .env.docker, and test.yml files > 2. Review the changes in the main.go file for HTTP delivery and service setup > 3. Run the integration scripts in the dev folder > 4. Check the new Dockerfile and README.md in the integration directory > 5. Test the TypeScript files for message handling and configuration > > ## Why make this change > - Improve Docker configuration and environment variable management > - Enhance integration testing setup and job organization > - Add new services and scripts for better testing and development > - Update code for enabling HTTP delivery and service setup > - Include new TypeScript files for message handling and configuration settings
neekolas commented 6 months ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @neekolas and the rest of your teammates on Graphite Graphite

neekolas commented 5 months ago

Merge activity