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

Running into an issue while trying to create tests for ruleset #9

Closed
zmshah opened this issue May 9, 2021 · 2 comments
Closed

Running into an issue while trying to create tests for ruleset #9

zmshah opened this issue May 9, 2021 · 2 comments

Comments

@zmshah
Copy link

zmshah commented May 9, 2021

I was trying to add tests to this ruleset similar to some of the other rulesets. Added a visual test runner and testsceneosugame but running into an issue when trying to create the player test scene. This is the error osu.Framework.Allocation.DependencyNotRegisteredException: 'The type SoyokazePlayfield has a dependency on SoyokazeConfigManager, but the dependency is not registered.'

I believe the problem is with the load method in soyokazeplayfield. I tried passing true into the dependencyloader to see what happens which is how I came to this conclusion. I've attached the exception details as txt files as well as screenshots. Not really sure how to go forward with this.

p.s My group and I are currently trying to write tests for an opensource project as part of out Software testing class in college, and we thought that maybe we could try setting up soyokaze for tests in a similar way to other custom rulesets. Of course if you don't want us to do so with this project we will definitely stop.
Exception details 1.txt
Exception details 2.txt
Screenshot 2021-05-09 00 06 46
Screenshot 2021-05-09 00 07 26
Screenshot 2021-05-09 00 19 27

@goodtrailer
Copy link
Owner

f5fc4c5 Allow null config manager with dependency loader
I'm hoping this commit fixed your issue. If not, then I'm not really sure how to fix it (the background dependency loader is honestly a bit magical to me). And feel free to do your project, this is open-source for a reason. I don't mind at all, and good luck. 👍

@zmshah
Copy link
Author

zmshah commented May 9, 2021

Thank you so much. We'll try it out!!

@zmshah zmshah closed this as completed May 9, 2021
@zmshah zmshah reopened this May 9, 2021
@zmshah zmshah closed this as completed May 15, 2021
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

No branches or pull requests

2 participants