AvailableAction.vue - Unit Test

[cazarov]

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.

PR type

• Bugfix
• Feature
• Refactoring
• Tests

What's new?

• PR closes AvailableAction.vue - Unit Test #2964
• I have fixed the import errors that were blocking unit testing AvailableAction.vue (and other components) by adding the new development dependency @rollup/plugin-graphql and by refactoring the wallet configuration files to use ES6 imports.
• I have added the new development dependency fake-indexeddb as it is required to make AvailableAction.vue work in the non-browser jsdom environment used for unit testing which does not have an IndexedDB implemention by default. To not slow down non-component file unit testing, I chose not to add loading it to vitest.config.js.
• As a starting point, I have added three unit tests for AvailableAction.vue that check correct button type rendering based on props. More work is needed for proper unit test coverage for this component.

Before submitting Pull Request, please make sure:

• My contribution builds clean without any errors or warnings
• I've merged recent default branch -- main and I've no conflicts
• I've tried to respect high code quality standards
• I didn't break any original functionality
• I've posted a screenshot of demonstrated change in this PR

Optional

• I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
• I've tested PR on mobile and everything seems works
• I found edge cases
• I've written some unit tests 🧪

Had issue bounty label?

• Fill up your KSM address: Payout

Community participation

• Are you at KodaDot Discord?

Screenshot

• My fix has changed something on UI; a screenshot is best to understand changes for others.

[netlify]

✅ Deploy Preview for koda-nuxt ready!

Name	Link
🔨 Latest commit	be2a1b4
🔍 Latest deploy log	https://app.netlify.com/sites/koda-nuxt/deploys/62fbc2fab059410008079340
😎 Deploy Preview	https://deploy-preview-3738--koda-nuxt.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[roiLeo]

Hey! would you mind changing logo name with something more meaningfull (CloverLogo, LedgerLogo, NovaLogo ...)

Tests pass

[cazarov]

Hey! would you mind changing logo name with something more meaningfull (CloverLogo, LedgerLogo, NovaLogo ...)

Hi! :) Sure no problem, just added a commit.

[roiLeo]

✅ code lgtm

[DeluneClaire]

Looks like code only, so i will just approve this right?

[codeclimate]

Code Climate has analyzed commit 0acf958 and detected 7 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Duplication	7

View more on Code Climate.

[petersopko]

tests are passing for me, maybe let's resolve conflicts, update from main and lfg :)

[petersopko]

✅ code lgtm

👀

[petersopko]

wfm

[petersopko]

@cazarov can you check out the DeepSource warnings please?

[cazarov]

@cazarov can you check out the DeepSource warnings please?

All fixed, was just sort-imports after the rule got added in #3718.

[petersopko]

pay 50 usd

[yangwao]

pay 50

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $50 @ 54.61 USD/KSM ~ 0.916 $KSM
🧗 J98SRy9JBahuoKgBUsTcLT3oYofoDj82fsWBsT9CaXBAmWb
🔗 0x98c0ce2462fd18efa627dc8440e87f6d3d2d1760477580a072e376395f81d5d4

🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues