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.

Sign up for GitHub

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

Jump to bottom

[UNIT] ItemBuilder needs tests #23

Closed
iceBear67 opened this issue Feb 27, 2022 · 3 comments
Closed

[UNIT] ItemBuilder needs tests #23

iceBear67 opened this issue Feb 27, 2022 · 3 comments
Assignees
@iceBear67
Labels
good first issue Good for newcomers need-test Indicate an module/function related to the issue should be tested. priority: high
Milestone
0.1.0-M1

Comments

@iceBear67
Copy link
Member

No description provided.

@iceBear67 iceBear67 added good first issue Good for newcomers priority: high need-test Indicate an module/function related to the issue should be tested. labels Feb 27, 2022
@iceBear67 iceBear67 added this to the 0.0.1 milestone Feb 27, 2022
@iceBear67 iceBear67 self-assigned this Feb 27, 2022
@iceBear67 iceBear67 mentioned this issue Feb 27, 2022
Closed
iceBear67 added a commit that referenced this issue Mar 6, 2022
@iceBear67
Fix: PrototypeDecorator hasn't delegate getHolder()
354149b 
Refactor: (ItemBuilder#register) Use Builder style code instead of constructor

Add UnitTest for ItemBuilder

This related to #5 #23
@iceBear67
Copy link
Member Author

Particularly done, Code coverage is pending to be inspected

@iceBear67
Copy link
Member Author

Recipes are broken

@iceBear67
Copy link
Member Author

Recipes are fixed.

The last thing to test is whether texture works, however that's a task for #13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iceBear67 iceBear67

Labels
good first issue Good for newcomers need-test Indicate an module/function related to the issue should be tested. priority: high
Projects
None yet
Milestone
0.1.0-M1
Development

No branches or pull requests
1 participant
@iceBear67