Merge pull request #748 from nervosnetwork/shaojunda-fix-dao-cache-update

fix: dao cache update

Author: shaojunda

app/controllers/api/v1/contracts_controller.rb

@@ -4,10 +4,10 @@ class ContractsController < ApplicationController
       def show
         raise Api::V1::Exceptions::ContractNotFoundError if params[:id] != DaoContract::CONTRACT_NAME
 
-        dao_contract = DaoContract.where(id: 1)
+        dao_contract = DaoContract.default_contract
         json =
           Rails.cache.realize(dao_contract.cache_key, version: dao_contract.cache_version, race_condition_ttl: 3.seconds) do
-            DaoContractSerializer.new(dao_contract.first).serialized_json
+            DaoContractSerializer.new(dao_contract).serialized_json
           end
 
         render json: json

test/controllers/api/v1/address_udt_transactions_controller_test.rb

@@ -200,7 +200,7 @@ class AddressUdtTransactionsControllerTest < ActionDispatch::IntegrationTest
         page_size = 5
         udt = create(:udt, published: true)
         address = create(:address, :with_udt_transactions, transactions_count: 30, udt: udt)
-        address_udt_transactions = address.ckb_udt_transactions(udt.id).order(block_timestamp: :desc).page(page).per(page_size)
+        address_udt_transactions = address.ckb_udt_transactions(udt.id).order("block_timestamp desc nulls last, id desc").page(page).per(page_size)
 
         valid_get api_v1_address_udt_transaction_url(address.address_hash, type_hash: udt.type_hash), params: { page: page, page_size: page_size }
         options = FastJsonapi::PaginationMetaGenerator.new(request: request, records: address_udt_transactions, page: page, page_size: page_size).call

test/controllers/api/v1/contracts_controller_test.rb

@@ -77,6 +77,14 @@ class ContractsControllerTest < ActionDispatch::IntegrationTest
 
         assert_equal response_json, response.body
       end
+
+
+      test "should update dao contract cache when dao related data changed" do
+        dao_contract = DaoContract.default_contract
+        cache_version = dao_contract.cache_version
+        dao_contract.touch
+        assert_not_equal cache_version, dao_contract.reload.cache_version
+      end
     end
   end
 end