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

Fixed phpunit tests #953

Merged
merged 1 commit into from
Apr 15, 2024
Merged

Conversation

josaphatim
Copy link
Member

No description provided.

@wangxiaoerYah
Copy link
Contributor

It is working.

@wangxiaoerYah
Copy link
Contributor

wangxiaoerYah commented Apr 15, 2024

Test CI -selenium

I added your modification manually and it worked properly???

@wangxiaoerYah
Copy link
Contributor

I noticed you guys removed the send test? Maybe it was working?

@josaphatim
Copy link
Member Author

I noticed you guys removed the send test? Maybe it was working?

Are you talking about Hm_Test_Output::test_send_response ?

@wangxiaoerYah
Copy link
Contributor

I noticed you guys removed the send test? Maybe it was working?

Are you talking about Hm_Test_Output::test_send_response ?

That should be it.

@josaphatim josaphatim merged commit 66b0868 into cypht-org:master Apr 15, 2024
@josaphatim
Copy link
Member Author

I noticed you guys removed the send test? Maybe it was working?

Are you talking about Hm_Test_Output::test_send_response ?

That should be it.

The test has not been deleted

@wangxiaoerYah
Copy link
Contributor

workflows

Do you think this would be better?

My run status

@josaphatim
Copy link
Member Author

workflows

Do you think this would be better?

My run status

@kroky can you please give your opinion?

@kroky
Copy link
Member

kroky commented Apr 16, 2024

I will comment in #954

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.

None yet

3 participants