Change:[DI-29624] - Use binary based formulas for bits rollup

[venkymano-akamai]

Description 📝

Use binary based formulas for bits rollup in cloudpulse

Changes 🔄

1. Change unitconversion.ts to use binary based formulas to rollup bits to Mb, Kb, Gb etc.,
2. Fix the UT's accordingly

Scope 🚢

Upon production release, changes in this PR will be visible to:

• All customers
• Some customers (e.g. in Beta or Limited Availability)
• No customers / Not applicable

Target release date 🗓️Next Release date

Preview 📷

This is a unit rollup change, no change in UI / UX

How to test 🧪

This is a unit rollup change, no change in UI / UX

Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support

• I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

• All tests and CI checks are passing
• TypeScript compilation succeeded without errors
• Code passes all linting rules

[linode-gh-bot]

Cloud Manager UI test results

🎉 866 passing tests on test run #5 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
0 Failing	866 Passing	11 Skipped	35m 48s

[bnussman-akamai]

Just to confirm: Is it expected that *Bps and *bps map to the same value now?

  it('Kbps vs KBps', () => {
    expect(convertValueToUnit(1024, 'Kbps')).toBe(1);
    expect(convertValueToUnit(1024, 'KBps')).toBe(1);
  });

[venkymano-akamai]

Yes...correct, we are moving towards a binary based formula

only thing is Bps is Bytes per second, bps is bit per second