Page MenuHomeDevCentral

Add command to validate configuration files
ClosedPublic

Authored by dereckson on Apr 9 2017, 14:36.
Tags
None
Referenced Files
F2905759: D949.id2419.diff
Tue, May 7, 22:42
Unknown Object (File)
Sun, May 5, 06:35
Unknown Object (File)
Sun, May 5, 06:14
Unknown Object (File)
Sun, May 5, 05:05
Unknown Object (File)
Sun, May 5, 00:34
Unknown Object (File)
Sat, May 4, 15:57
Unknown Object (File)
Fri, May 3, 21:58
Unknown Object (File)
Fri, May 3, 21:55
Subscribers
None

Details

Summary

Currently, the command validates JSON syntax,
using the own PHP parser instead to rely on jsonlint,
to avoid to depends of Node.JS for one command.

Fixes T1177.

Test Plan

New tests check command output is expected when:

  • all the files are valid
  • there is a general syntax error
  • there is a specific syntax error

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson edited the test plan for this revision. (Show Details)
dereckson edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Apr 9 2017, 14:38
dereckson edited the test plan for this revision. (Show Details)

Style

This revision was automatically updated to reflect the committed changes.