1/2 threads resolved
- Resolved by Ben Hayward
marked as a Work In Progress
unmarked as a Work In Progress
changed the description 2 times within 4 minutes
resolved all discussions
- Last updated by Ben Hayward
79 79 expect(transformedString).toContain('<a class="tag"'); 80 80 }); 81 81 82 it('should transform when @ followed by `.com`', () => { 83 const pipe = new TagsPipe(featuresServiceMock); 84 const string = 'textstring @name.com'; 85 const transformedString = pipe.transform(<any>string); 86 expect(transformedString).toContain('<a class="tag"'); 87 expect(transformedString).toContain('@name.com'); 88 }); 89 90 it('should transform two adjacent tags', () => { - Owner
I think this test still needs more. Production bug had 12 tags in.
- Developer
No problem, I'll update now
added MR::Requires Changes label
added MR::Awaiting Review label and removed MR::Requires Changes label