search

udacity / nd1309-p2-Decentralized-Star-Notary-Service-Starter-Code

Code for the ND1309 v2 C2 project
Other
9 stars 97 forks source link

Fix confusing test #14

Closed samraul closed 2 years ago

samraul commented 2 years ago

Fix confusing test (see issue #13) by requiring properties to be set in the constructor of the contract.

jungleBadger commented 2 years ago

Hey there, @samraul. Thanks for your contribution.

I kindly disagree that this modification is needed since creating a new Star can be omitted in that step. Stars made in the previous test cases in that test suite can be used as a reference for validating their Name and Symbol.