Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate failing integration tests #231

Closed
Mzack9999 opened this issue Mar 13, 2023 · 0 comments · Fixed by #251
Closed

Investigate failing integration tests #231

Mzack9999 opened this issue Mar 13, 2023 · 0 comments · Fixed by #251
Assignees
Labels
Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.
Milestone

Comments

@Mzack9999
Copy link
Member

Notify version:

dev

Current Behavior:

https://github.com/projectdiscovery/notify/actions/runs/4394624953/jobs/7695782120

Run chmod +x action-run.sh
Running test cases for "discord"
[✘] Test "discord" failed: incorrect number of results 
             __  _ ___    
  ___  ___  / /_(_) _/_ __
 / _ \/ _ \/ __/ / _/ // /
/_//_/\___/\__/_/_/ \_, / v1.0.4
                   /___/  
		projectdiscovery.io
Use with caution. You are responsible for your actions
Developers assume no liability and are not responsible for any misuse or damage.
hello from notify integration test :)
[ERR] incorrect discord configuration for id: disocord-integration-test 
Running test cases for "slack"
[✘] Test "slack" failed: incorrect number of results 
             __  _ ___    
  ___  ___  / /_(_) _/_ __
 / _ \/ _ \/ __/ / _/ // /
/_//_/\___/\__/_/_/ \_, / v1.0.4
                   /___/  
		projectdiscovery.io
Use with caution. You are responsible for your actions
Developers assume no liability and are not responsible for any misuse or damage.
hello from notify integration test :)
[ERR] failed to send slack notification for id: slack-integration-test : invalid slack token format
Running test cases for "custom"
[✘] Test "custom" failed: incorrect number of results 
             __  _ ___    
  ___  ___  / /_(_) _/_ __
 / _ \/ _ \/ __/ / _/ // /
/_//_/\___/\__/_/_/ \_, / v1.0.4
                   /___/  
		projectdiscovery.io
Use with caution. You are responsible for your actions
Developers assume no liability and are not responsible for any misuse or damage.
hello from notify integration test :)
[ERR] failed to send custom notification for id: custom-integration-test : Post "": unsupported protocol scheme ""
Error: Process completed with exit code 1.

Expected Behavior:

Successful integration tests

Steps To Reproduce:

Issue any PR on dev branch

@Mzack9999 Mzack9999 added the Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors. label Mar 13, 2023
@RamanaReddy0M RamanaReddy0M self-assigned this May 4, 2023
@RamanaReddy0M RamanaReddy0M linked a pull request May 4, 2023 that will close this issue
@ehsandeep ehsandeep added the Status: Completed Nothing further to be done with this issue. Awaiting to be closed. label May 4, 2023
@ehsandeep ehsandeep added this to the notify v1.0.5 milestone May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants