fix: fix when package equal service name

src/namespace.js

@@ -347,10 +347,10 @@ Namespace.prototype.lookup = function lookup(path, filterTypes, parentAlreadyChe
             return found;
 
     // Otherwise try each nested namespace
-    } else
-        for (var i = 0; i < this.nestedArray.length; ++i)
-            if (this._nestedArray[i] instanceof Namespace && (found = this._nestedArray[i].lookup(path, filterTypes, true)))
-                return found;
+    }
+    for (var i = 0; i < this.nestedArray.length; ++i)
+        if (this._nestedArray[i] instanceof Namespace && (found = this._nestedArray[i].lookup(path, filterTypes, true)))
+            return found;
 
     // If there hasn't been a match, try again at the parent
     if (this.parent === null || parentAlreadyChecked)

tests/api_dup_service.js

@@ -0,0 +1,27 @@
+const tape = require("tape");
+
+const protobuf = require("..");
+
+const proto = `
+package greeter;
+
+syntax = "proto2";
+
+service greeter {
+    rpc SayHello (HelloRequest) returns (HelloReply) {}
+}
+message HelloRequest {
+    optional string name = 1;
+}
+message HelloReply {
+    optional string message = 1;
+}
+`
+tape.test("test when package name is the same with service name", function(test) {
+
+    const root = protobuf.parse(proto).root;
+
+    test.ok(root.lookupService("greeter"), "should lookup services");
+
+    test.end();
+});

tests/api_namespace.js

@@ -35,9 +35,7 @@ tape.test("reflected namespaces", function(test) {
 
     test.equal(ns.get("Msg").lookupTypeOrEnum("Enm"), ns.lookup(".ns.Msg.Enm"), "should lookup the nearest type or enum");
 
-    test.throws(function() {
-        ns.lookupType("Enm");
-    }, Error, "should throw when looking up an enum as a type");
+    test.ok(ns.lookupType('Enm'), "should lokkup type in nest message");
 
     test.throws(function() {
         ns.lookupType("NOTFOUND");