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

Write tests #7

Open
Stantheman opened this issue Dec 9, 2013 · 3 comments
Open

Write tests #7

Stantheman opened this issue Dec 9, 2013 · 3 comments

Comments

@Stantheman
Copy link
Owner

fuse-colors would have been a nightmare to write tests for, but this should be pretty straightforward and would be good to do. It would be cool if I was a TDD guy, but this is a cool first step.

@Stantheman
Copy link
Owner Author

I have a giant OPML file from newsblur since my account expired that I can test with now. Both happy and sad

@Stantheman
Copy link
Owner Author

I've been using it for testing but haven't written real tests. I definitely need them. I guess if I was going to go with trying to be a good person, I'd also stop shoving this into one file. I just think it looks cuter that way.

Should probably split it into a real thing and then fatpack it

@Stantheman
Copy link
Owner Author

I feel bad because I either need to hard-code a few URLs and hope they don't go away, or I need to skip the LWP part and fake it with locally accessible data. Although Moonpig recently fought for this kind of thing (Faking the provider of a function for testing's sake), I don't think I'm in the same situation. I'm actively using this for 85 feeds every five minutes, and have tested it with ~400 feeds.

I'm probably making excuses at this point, but I'm torn about testing.

I'm pretty sure I'm making excuses. I can keep the data local. Still sad.

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

1 participant