Skip to content

Commit

Permalink
Merge pull request #17 from simeji/fix-cli-option
Browse files Browse the repository at this point in the history
remove -p option
  • Loading branch information
simeji authored Nov 13, 2016
2 parents 29f7408 + b243b91 commit 6e123b1
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 18 deletions.
2 changes: 0 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,4 @@ curl -s http://rdg.afilias.info/rdap/domain/example.info | jig

### Option

-p : Pretty print (output json)

-q : Print query (for jq)
10 changes: 10 additions & 0 deletions cmd/coverage.out
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
mode: set
github.com/simeji/jig/cmd/jig.go:10.13,22.2 8 0
github.com/simeji/jig/cmd/jig.go:24.62,27.30 2 1
github.com/simeji/jig/cmd/jig.go:30.2,30.11 1 1
github.com/simeji/jig/cmd/jig.go:42.2,42.10 1 1
github.com/simeji/jig/cmd/jig.go:27.30,29.3 1 1
github.com/simeji/jig/cmd/jig.go:30.11,32.3 1 1
github.com/simeji/jig/cmd/jig.go:32.3,32.19 1 1
github.com/simeji/jig/cmd/jig.go:32.19,38.3 0 1
github.com/simeji/jig/cmd/jig.go:38.3,41.3 2 1
15 changes: 3 additions & 12 deletions cmd/jig.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,33 +11,24 @@ func main() {
content := os.Stdin

var query bool
var pretty bool

flag.BoolVar(&query, "q", false, "output query")
flag.BoolVar(&pretty, "p", false, "pretty print")
flag.Parse()

e := jig.NewEngine(content)
os.Exit(run(e, query, pretty))
os.Exit(run(e, query))
}

func run(e jig.EngineInterface, query bool, pretty bool) int {
func run(e jig.EngineInterface, query bool) int {

result := e.Run()
if result.GetError() != nil {
return 2
}
if query {
fmt.Printf("%s", result.GetQueryString())
} else if pretty {
//s, _, _, err := e.manager.GetPretty(e.query, true)
//if err != nil {
//return 1
//}
//fmt.Printf("%s", s)
} else {
s := result.GetContent()
fmt.Printf("%s", s)
fmt.Printf("%s", result.GetContent())
}
return 0
}
8 changes: 4 additions & 4 deletions cmd/jig_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,22 +20,22 @@ func TestJigRun(t *testing.T) {
var assert = assert.New(t)

e := &EngineMock{err: nil}
result := run(e, false, false)
result := run(e, false)
assert.Zero(result)
assert.Equal(2, called)

result = run(e, true, false)
result = run(e, true)
assert.Equal(1, called)

result = run(e, false, true)
result = run(e, false)
assert.Zero(result)
}

func TestJigRunWithError(t *testing.T) {
called = 0
var assert = assert.New(t)
e := &EngineMock{err: fmt.Errorf("")}
result := run(e, false, false)
result := run(e, false)
assert.Equal(2, result)
assert.Equal(0, called)
}
Expand Down
1 change: 1 addition & 0 deletions engine.go
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,7 @@ func (e *Engine) Run() EngineResultInterface {
cc, _, _, err := e.manager.Get(e.query, true)
return &EngineResult{
content: cc,
qs: e.query.StringGet(),
err: err,
}
}
Expand Down

0 comments on commit 6e123b1

Please sign in to comment.