1. Useless, general phrases saying nothing about what method is actually doing:
it "adds certain value"
it "returns correct result"
it "fails"
it "returns formatted string"
it "returns correct url"
it "is ok"
2. Implementation details:
it "changes @scheduled_on"
it "assigns @todos"
3. Lies:
it "returns time in 24-hour format" do
expect(...).to eq "9:25"
end
it "strips leading zeroes" do
expect(foo(" 9:25 ")).to eq "9:25"
end
Top comments (0)