Testing Linode Power #1
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| } from '../../../src/actions/api/linodes'; | ||
| import * as linode_actions from '../../../src/actions/api/linodes'; | ||
|
|
||
| const mock_context = async (f, rsp, state={}) => { |
There was a problem hiding this comment.
Is there any reason not to share this with the identical function in the api-store spec?
There was a problem hiding this comment.
👍 should be moved to some shared code, tests/mocks.js perhaps.
This was referenced Jun 9, 2017
3 tasks
Jskobos
added a commit
that referenced
this pull request
Mar 27, 2019
* M3-2578 Fix config updating bug This turned out to be two separate issues. 1. Our createDevicesFromStrings utility was returning "none_id: null" for the disk ID instead of null as it should have been. 2. There is a small API bug where POST requests for creating a config were not validated, whereas PUT requests were. This results in a state where it is possible to create a config that then can't be updated with its initial values. Fixed #1 by adding a === 'none' check in createDevicesFromStrings, added temporary client-side validation for #2. * Add test case for devices set to None * Update copy
martinmckenna
pushed a commit
that referenced
this pull request
Mar 29, 2019
* M3-2578 Fix config updating bug This turned out to be two separate issues. 1. Our createDevicesFromStrings utility was returning "none_id: null" for the disk ID instead of null as it should have been. 2. There is a small API bug where POST requests for creating a config were not validated, whereas PUT requests were. This results in a state where it is possible to create a config that then can't be updated with its initial values. Fixed #1 by adding a === 'none' check in createDevicesFromStrings, added temporary client-side validation for #2. * Add test case for devices set to None * Update copy
benschmaus
pushed a commit
to benschmaus/manager
that referenced
this pull request
Apr 1, 2019
* M3-2578 Fix config updating bug This turned out to be two separate issues. 1. Our createDevicesFromStrings utility was returning "none_id: null" for the disk ID instead of null as it should have been. 2. There is a small API bug where POST requests for creating a config were not validated, whereas PUT requests were. This results in a state where it is possible to create a config that then can't be updated with its initial values. Fixed linode#1 by adding a === 'none' check in createDevicesFromStrings, added temporary client-side validation for linode#2. * Add test case for devices set to None * Update copy
patthiel
pushed a commit
that referenced
this pull request
Apr 1, 2019
* Configure non-field errors for NodeBalancers (#4672) * Configure non-field errors for NodeBalancers * Fix scrolling into an error * M3-OBJ-56 Change: Create Buckets (#4706) * bucket drawer in redux * clusters in redux * create bucket methods and functionality * make bucketDrawer its own container * clean * add tests * remove success toast * do not remove any attributes (#4707) * Support for storing credentials in MongoDB * Adds CredStore support for cleaning up after tests * Cleans up clean up code and fixes initialization for mongo * Tweaks for mongo cleanup code. Better differentiates between failure deleting resources via API vs failure cleaning up mongo. * refactors code that cleans up when all tests finish. * Fixes cleanup for users and oauth clients * Updates variable assignments to use constants per feedback from Pat * Refactors to remove un-needed Promise wrappers and uses async fs methods for FSCredStore * Add mongo service * updates functions that read the logged in user's access token in config-utils * using FSCredStore to compare to MongoCredStore running on baker * Add event types and account events messages (#4692) * fix nb create crashing (#4683) * M3-2568 add missing typography for crash message * M3-2565 Change in_progress event polling interval back to 1s (#4691) * M3-2570 Fix: Only disable Linode Boot if there are no configs (#4696) * use linode's configs instead of image to determine "Power On" visibility * fix loading icon placement and styling * remove if found logic (#4700) * Test Maintenance (#4699) * StackScripts e2e test maintenance (#4698) * Fix PAT Token Test (#4697) * first commit * Refactor, jsClick action menu if fails to standard click * unskip desk * refresh upon deleting token * add data-qa to backdrop * change jasmine timeout back to 10 * Rebuild E2E Test Maintenance (#4695) * rebuild test fixes * respond to feeback * Fix Volumes and Domains tests (#4694) * Fix Volumes and Domains tests * fix volumes tags test * Increase wait time when attaching a volume to wait.long * M3-2569-timezone-tests fix test * fix remove linode test (#4702) * update toast message (#4701) * v0.51.0 * Update CHANGELOG.md * read IS_OBJECT_STORAGE_ENABLED as a string * v0.51.1 * fix lish route (#4736) * v0.51.2 * update changelog * M3-OBJ Update Object Storage CTA (#4716) * update CTA to include bucket creation * improve cluster select form * M3-2578 Fix config updating bug (#4712) * M3-2578 Fix config updating bug This turned out to be two separate issues. 1. Our createDevicesFromStrings utility was returning "none_id: null" for the disk ID instead of null as it should have been. 2. There is a small API bug where POST requests for creating a config were not validated, whereas PUT requests were. This results in a state where it is possible to create a config that then can't be updated with its initial values. Fixed #1 by adding a === 'none' check in createDevicesFromStrings, added temporary client-side validation for #2. * Add test case for devices set to None * Update copy * Feature/cloud apps (#4590) * add CA create Linode skeleton (#4577) * CA-XXX Add Distros tab (#4579) * Working prototype * Just refactor FIC after all * add subtabs (#4581) * CA: Add my images tab (#4582) * Refactor clone from linode tab to new structure * Add CASelectStackScriptPanel * lift state up and fix fromimagecontent * fix linode clone view * fix bug * destructure unnecessary props * Working mostly * Remove console log * use container form state in createfromstackscript * Add Cloudapps icons to src/assets (#4587) * fix tabs render bug (#4592) * Reset creation state on tab change (#4595) * add error state to creation flow (#4596) * CA: Move SSH HOC to container (#4597) * Move SSH key HOC to container * Reset selected SSH keys correctly * CLAPPS - Tab State in Redux (#4598) * set create flow type in redux state * do it better this time * bad spelling * CLAPPS: Add Images subtab (#4599) * WIP * WIP 2 * Fix styling * Fix StackScripts to use new SelectImagePanel * Fix tab labels * Add empty state for private images * Review feedback - Combine renderPublicImages and renderOlderPublicImages into a single function - Use <span> instead of <Typography> for placeholder message - Only pass needed props to FromImageContent * Pass fewer props to other FromImagesContent instance * Clean up post-rebase * CLAPPS - fix dynamic label (#4601) * fix dynamic label logic * fix logic * address feedback * CLAPPS Add create from backups tab (#4603) * WIP * Everything working except errors * Functional * Trim unused props * Fix backup client-side validatioN * Review feedback * Utilities has 3 i's * Use dynamic value for isMakingRequest in FromBackupsContent * CA-606 Nested Tab Panel Updates (#4586) * adding paper to the inner tabvs to match pattern as existing panels * fixing imports * removing redundant classes that were messing up grid alignment * fixing broken sidebar * updating backups tab ordering * CLAPPS: Add Community StackScripts panel (#4612) * Add Community StackScripts panel * Review feedback -- adjust styling * CLAPPS - Create from App (#4604) * init commit * cleanup * init createapppanel * add apps panel * update label logic * move apps request to container * reset errors on new stackscript selection * fix placement of closing grid tag * fix rebase issues * CLAPPS Map UDF error fields (#4619) * Map UDF errors to the correct field * Fix Notice spacing for select and multiselect UDFs * Apps UI Edits (#4611) * adding errors to linodeContent panel instead of component * fixing gaping issue * removing sticky stuff from checkout component, adding sticky css * adding more vertical spacing for larger screensizes * adding conditional based on image variant public vs private * style adjustments for apps content * cleanup from moving notices * CLAPPS: Fix StackScript routing (#4625) * Fix typing * Fix routing (was reloading full site) * Review feedback * M3-2512 Move Apps Logos (#4633) * move images into public dir * remove logos from src dir * M3 clapps icons (#4644) * Add hard-coded icons * Adapt for Linode account request * Format Clapps names for display * Clean up 2 * Review feedback - Add APP_ROOT constant to icon URL * Remove duplicate URL slash * Apps UDF Improvements (#4629) * putting optional fields into collapsible container * actually filtering properly now * some styling edits * removing test styles * applying grid to optional form UDFs * applying max width * removing grid from individual udf fields * improving form grids for other input types, adding prop for password input to hide strength span * fix keys errors and remove inconsistent margins * adding key to grid items * making optional fields open by default if no required fields, adjustments to other components using showMoreExpansion component * addressing PR feedback * additional updates * leverage one-click-apps endpoint (#4659) * pass down stackscript id (#4668) * Update Rust icon (#4671) * M3 Fix clapps advanced option bug (#4669) * Update cDU logic Added a check to cDU so that if the default value of defaultOpen (on ExpansionPanel) changed, the view would update, but would not update if the value was re-computed to be the same as the current value. Fixes a bug where if the user typed in the Advanced Options fields, the panel would close. * Fix bug * CLAPPS: Add defaultImage logic (#4674) * Add defaultImage logic * Remove additional is-this-an-app? logic * Cloud Apps - Bug Fixes (#4677) * bug fixes * Copy fixes * adding missing class to empty linodes to clone container (#4688) * appease tests and correct types * fix some more typings * fix paginated select linode panel * un-skip tests * cleanup tests * remove capuk helper * fix RenderGuard issues * fix typography issues on create from image * defaulting expansion panel for optional udfs to be open * Rename files to remove CA convention (#4714) * Remove creation type from auto-generated label (#4715) * [Cloud Apps] Unit Testing (#4708) * add unit tests * typo * destructure props (#4713) * One-click app feedback (#4720) * Review feedback part 1 - Fixed shadowed variable name images in PublicImages.tsx - Re-added Alpine to PublicImages.tsx - Update error message to use new One-Click Apps branding * Fix routing when deploying from a backup * [Cloud Apps] - StackScript Updates (#4709) * refactor stackscript logic * update get account stackscripts to get current user * fixes * Fix spelling * add util function for unauthed errors * remove tab handling from selectstackscriptpanel (#4721) * refactor smoke test (#4728) * M3-2579 add loading states to tag panel * OCA: fix image routing (#4727) * Update routing for creating from image * Set search directly in location * Error handling and styling * add front end validation for tag input error * cleanup * address feddback * Merge pull request #4717 from alioso/M3-2579 M3-2579 Improve TagPanel workflow * M3-2593: fix restore process finished event handling (#4740) * Control which cred store gets used by setting MONGO_HOST env var * Add Mongo host env var * Fix ternary * Actually fix ternary * rename mongo_host to CRED_STORE_MODE and refactor ternary again
Jskobos
added a commit
that referenced
this pull request
Jun 19, 2019
* M3-2899 Fix: Catch deleted events errors After a Linode was deleted, if it had long-running events on it, our linode.events.ts logic was requesting the (nonexistent) Linode after every event, and these errors were uncaught and therefore being reported to Sentry. Fixed this in two ways: - Added a catch block to the requestLinodeForStore method, which drops the error - Added a check in the requestLinodeForStore thunk, so that the API is only hit for a Linode that exists in our store. Since requestLinodeForStore is only called for existing Linodes, this shouldn't cause any issues; however, fix #1 above is enough to resolve this issue, so it might be safer to remove this. * Return a Promise just to be safe
acourdavault
added a commit
that referenced
this pull request
Jan 6, 2020
Change uuid prop to key to work with Notistack 0.9.7
Merged
johnwcallahan
added a commit
that referenced
this pull request
Feb 6, 2020
Previously Pull Request IDs were included on each commit message. Example: "My first commit (#1)". These had to be removed manually, so this saves a few steps. I also cleaned up the way the lines were being read from standard output.
This was referenced Jul 21, 2020
Jskobos
added a commit
that referenced
this pull request
Oct 1, 2020
* Fix key warning * Add form fields to create modal * Add VLAN context for opening/closing modal; move modal to main content level * Clean up * Add initial unit tests * Add updated cache for regions data * Error handling and tests - Fix/finish unit tests - Add Notice for displaying general errors - Clear form when the modal opens * Review feedback #1 - Adjust spacing in create modal - Don't set a default region (we don't do that elsewhere and our alphabetical-by-country sorting makes it hard to dynamically choose a stable default) - Adjust schema now that description has a limit of 64 chars * Clear checkbox when Linode select is cleared * Make dialog context generic * Fix Linode filtering by region - Update LinodeMultiSelect to allow it to optionally be controlled from its parent. (it now takes selectedLinodes, a number[] which matches the signature of handlechange). - In Vlan create, if no region has been selected, filter Linodes by all allowed regions (any region with capabilities.vlans). If a region is selected, a) clear the selected Linodes and b) filter the Linode select by the target region. * Select a Linode's region when selecting a Linode (and region hasn't already been selected) * Finalize logic When creating a VLAN: 1. If the user selects a region first (normal pattern), filter the Linode select by that region. 2. If the user leaves region blank and first selects a region, automatically select that Linode's region (and consequently filter the Linode select by that region, so subsequent Linodes are all in the same region). 3. If a Linode is selected and the region is toggled, clear the selected Linode if it's not in the newly selected region. (Since we auto-select the region when the Linode is first selected, any toggle of the region input can be assumed to be in a different region than the currently selected Linodes). 4. Reset the checkbox (for reboot after create) whenever the list of Linodes is reset or manually cleared for any reason. * Fix tests
Merged
3 tasks
1 task
This was referenced Apr 21, 2023
This was referenced Sep 7, 2023
20 tasks
This was referenced Sep 13, 2023
This was referenced Sep 14, 2023
This was referenced Sep 29, 2023
This was referenced Oct 4, 2023
12 tasks
12 tasks
12 tasks
bnussman-akamai
added a commit
that referenced
this pull request
Aug 8, 2024
…noreDeprecations` Typescript Options (#10755) * save changes #1 * save changes #2 * save changes #3 * save changes #4 * save changes #5 * save changes #6 * save changes #7 * save changes #8 * fix image select * save changes #9 * save changes #10 * last issue is filtering * fix unit test by fixing button types * finally resolved all tsc errors * Added changeset: Remove `suppressImplicitAnyIndexErrors` and `ignoreDeprecations` Typescript Options * fix cypress feature flag util hopefully * another attemnpt to fix cypress feature flag util function * make image select close onSelect to match previous behavior * feedback @jaalah-akamai --------- Co-authored-by: Banks Nussman <banks@nussman.us>
ankita-akamai
referenced
this pull request
in ankita-akamai/manager
Dec 2, 2024
[DI-21597]-Initial commit: Automating and testing the migration process for Recharts
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.