Page MenuHomeDevCentral

Add command to validate configuration files
ClosedPublic

Authored by dereckson on Apr 9 2017, 14:36.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 18, 09:13
Unknown Object (File)
Sat, May 18, 09:09
Unknown Object (File)
Sat, May 18, 09:06
Unknown Object (File)
Sat, May 18, 00:14
Unknown Object (File)
Tue, May 14, 03:01
Unknown Object (File)
Mon, May 13, 13:51
Unknown Object (File)
Mon, May 13, 13:27
Unknown Object (File)
Mon, May 13, 11:45
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 Passed
Unit
Tests Passed
Branch
validate-json
Build Status
Buildable 1469
Build 1717: arc lint + arc unit

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.