search

ZeroKnight / ZeroBot-Perl

[OLDER INCARNATION] Personal use IRC bot intended solely for amusement. A pet project of sorts...
MIT License
0 stars 1 forks source link

Rewrite parser.t to not use ZeroBot::Command objects #11

Closed ZeroKnight closed 6 years ago

ZeroKnight commented 6 years ago

Being the first test for ZeroBot that I wrote, this is a growing pain from my still ongoing adventures into becoming familiar with writing unit tests. Testing Command::Parser should not involve creating Command objects, but instead in Commands own tests.

ZeroKnight commented 6 years ago

Actually, no. The current design leaves the two necessarily coupled. Command::Parser takes a Command object, so I don't believe there's much problem with the test as-is.

However, we'll see if this sentiment survives any potential refactors/redesigns.