fixing rns expired results in query (#496)

JackalLabs · Dec 12, 2024 · 8bed924 · 8bed924
2 parents 822bec0 + 2416270
commit 8bed924
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/scripts/upgrade-test.sh b/scripts/upgrade-test.sh
@@ -16,7 +16,7 @@ export GOMODCACHE=${ROOT}/../_build/gocache
 
 mkdir -p ../_build/old
 git checkout $OLD_VERSION
-make build
+go build -mod=readonly -o build/canined ./cmd/canined
 mv build/canined ./../_build/old/canined
 git checkout $NEW_VERSION
 make install

diff --git a/x/rns/keeper/grpc_query_list_owned_names.go b/x/rns/keeper/grpc_query_list_owned_names.go
@@ -53,8 +53,10 @@ func (k Keeper) ListOwnedNames(goCtx context.Context, req *types.QueryListOwnedN
 		}
 
 		if names.Value == req.Address {
-			namess = append(namess, names)
-			i++
+			if names.Expires > ctx.BlockHeight() {
+				namess = append(namess, names)
+				i++
+			}
 		}
 
 	}

diff --git a/x/rns/keeper/register_test.go b/x/rns/keeper/register_test.go
@@ -75,6 +75,11 @@ func (suite *KeeperTestSuite) TestMsgRegisterName() {
 		suite.Require().NoError(err)
 	}
 
+	rrr, err := suite.queryClient.AllNames(suite.ctx.Context(), &types.QueryAllNames{})
+	suite.Require().NoError(err)
+
+	suite.Require().Equal(100, len(rrr.Name))
+
 	r := types.QueryListOwnedNames{
 		Address: address.String(),
 	}