Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 447445

Summary:	[CSSLint] We should have more than one css validator test
Product:	[ECD] Orion	Reporter:	Curtis Windatt <curtis.windatt.public>
Component:	Client	Assignee:	Curtis Windatt <curtis.windatt.public>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	6.0
Target Milestone:	7.0
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Curtis Windatt

2014-10-15 15:00:25 EDT

Right now there is a single basic test in cssValidatorTests.js, we should have a suite of tests for the following:

1) CSSLint runs and finds problems
2) Default for all problems is warning
3) Parsing errors are marked as errors
4) Changing a severity settings results in correct severity
5) Embedding a ruleset overrides defaults and user settings
6) Having a space at start of embedded ruleset is allowed
7) With multiple embedded rulesets, only the first is used, the rest ignored.

Comment 1 Curtis Windatt

2014-10-16 15:36:16 EDT

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=4be255f7cd1e08dee795231300f358718f29e7dc
Added tests for everything except user settings

Comment 2 Curtis Windatt

2014-10-23 13:40:16 EDT

Not planning on doing tests for user settings in 7.0.  Marking as FIXED.