fix: validate rules in Apisix Dashboard is same as Apisix api

[iamayushdas]

Please answer these questions before submitting a pull request, or your PR will get closed.

Why submit this pull request?

• Bugfix
• New feature provided
• Improve performance
• Backport patches

What changes will this PR take into?

Please update this section with detailed description.

Related issues

fix/resolve #2057

Checklist:

• Did you explain what problem does this PR solve? Or what new features have been added?
• Have you added corresponding test cases?
• Have you modified the corresponding document?
• Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

[netlify]

✔️ Deploy Preview for apisix-dashboard ready!

🔨 Explore the source changes: 791647a

🔍 Inspect the deploy log: https://app.netlify.com/sites/apisix-dashboard/deploys/6127028e0cfc9f00077ad242

😎 Browse the preview: https://deploy-preview-2085--apisix-dashboard.netlify.app

[iamayushdas]

cc @liuxiran @guoqqqi @juzhiyuan

[liuxiran]

marked an unstable test case here: https://github.com/apache/apisix-dashboard/pull/2085/checks?check_run_id=3410472747

[liuxiran]

Hi @iamayushdas , would you please add test case for your changes?

just add assertion to check a route name with 101 characters can be added to this test case,

apisix-dashboard/web/cypress/integration/route/create-edit-duplicate-delete-route.spec.js

Line 86 in fb78963

it('should create route', function () {

[iamayushdas]

Sure sure

[codecov-commenter]

Codecov Report

Merging #2085 (914c65c) into master (fb78963) will decrease coverage by 3.81%.
The diff coverage is n/a.

❗ Current head 914c65c differs from pull request most recent head 791647a. Consider uploading reports for the commit 791647a to get more accurate results

@@            Coverage Diff             @@
##           master    #2085      +/-   ##
==========================================
- Coverage   68.87%   65.05%   -3.82%     
==========================================
  Files         181      121      -60     
  Lines        7010     3231    -3779     
  Branches      787      787              
==========================================
- Hits         4828     2102    -2726     
+ Misses       1898     1129     -769     
+ Partials      284        0     -284     

Flag	Coverage Δ
backend-e2e-test	?
backend-e2e-test-ginkgo	?
backend-unit-test	?
frontend-e2e-test	65.05% <ø> (-1.58%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
web/src/pages/Route/components/Step1/MetaView.tsx	100.00% <ø> (ø)
...pages/Route/components/Step1/MatchingRulesView.tsx	26.13% <0.00%> (-38.64%)	⬇️
web/src/pages/Route/transform.ts	72.41% <0.00%> (-7.48%)	⬇️
web/src/pages/Route/Create.tsx	82.92% <0.00%> (-1.63%)	⬇️
web/src/pages/Route/List.tsx	82.84% <0.00%> (-1.19%)	⬇️
api/internal/utils/closer.go
api/internal/filter/schema.go
api/internal/handler/label/label.go
api/internal/core/storage/storage_mock.go
api/internal/handler/consumer/consumer.go
... and 55 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb78963...791647a. Read the comment docs.

[iamayushdas]

@liuxiran its done

[guoqqqi]

LGTM!