there should be at least one test if only one parameter is empty like
```
context "with empty argument" do
  it "returns false" do
    expect(@iscsilib.ipEqual?("", "")).to be_false
    expect(@iscsilib.ipEqual?("127.0.0.1:22", "")).to be_false
    expect(@iscsilib.ipEqual?("", "127.0.0.1:22")).to be_false
```

and missing space after `,` it really helps readability