Merge pull request #153 from poap-xyz/bugfix/flaky-test-break-iterabl…

…e-symbol

Bugfix/flaky test break iterable symbol

cypress/support/e2e.js

@@ -59,10 +59,19 @@ export const check_if_code_is_expected = ( code_to_check, expected_codes ) => {
 
 export async function extract_redirect_url ( response ) {
 
-    cy.log( `Url from which to extract challenge: `, response )
-    // strinified response
-    console.log( `Response data: `, JSON.stringify( response ) )
+    cy.log( `Response object from which to extract challenge (can be printed to browser console): `, response )
     const { redirects } = response
+
+    // If there are no redirects, something weird happens that we need to debug
+    if( !redirects?.length ) {
+        cy.log( `No redirects found, full response body (status ${ response.status }): `, response.body )
+        cy.log( `There were ${ response?.allRequestResponses?.length } responses to the request:` )
+        response?.allRequestResponses?.forEach( unique_res => {
+            cy.log( `Status ${ unique_res[ 'Response Status' ] } from ${ unique_res[ 'Request URL' ] }`, unique_res[ 'Response Body' ] )
+        } )
+        return null
+    }
+
     const [ redirect_url ] = redirects
     cy.log( `Redirect: `, redirect_url )
     return redirect_url

functions/index.js

@@ -27,14 +27,14 @@ exports.getEventDataFromCode = v1_oncall( getEventDataFromCode )
 exports.check_code_status = v1_oncall( check_code_status )
 
 // Refresh all codes ( trigger from frontend on page mount of EventView )
-exports.requestManualCodeRefresh = v1_oncall( [ 'high_memory', 'long_timeout'  ], refresh_unknown_and_unscanned_codes )
+exports.requestManualCodeRefresh = v1_oncall( [ 'memory_512MiB', 'long_timeout'  ], refresh_unknown_and_unscanned_codes )
 
 // Allow frontend to trigger updates for scanned codes, ( triggers on a periodic interval from EventView ), is lighter than requestManualCodeRefresh as it checks only scanned and claimed == true codes
-exports.refreshScannedCodesStatuses = v1_oncall( [ 'high_memory', 'long_timeout' ], refreshScannedCodesStatuses )
+exports.refreshScannedCodesStatuses = v1_oncall( [ 'memory_512MiB', 'long_timeout' ], refreshScannedCodesStatuses )
 
 // Directly mint a code to an address
 const { mint_code_to_address } = require( './modules/minting' )
-exports.mint_code_to_address = v2_oncall( [ 'high_memory', 'long_timeout' ], mint_code_to_address )
+exports.mint_code_to_address = v2_oncall( [ 'memory_512MiB', 'long_timeout' ], mint_code_to_address )
 
 // Let admins recalculate available codes
 const { recalculate_available_codes_admin } = require( './modules/codes' )
@@ -45,7 +45,7 @@ exports.recalculate_available_codes = v2_oncall( recalculate_available_codes_adm
 // ///////////////////////////////
 
 const { registerEvent, deleteEvent, getUniqueOrganiserEmails } = require( './modules/events' )
-exports.registerEvent = v1_oncall( [ 'high_memory', 'long_timeout' ], registerEvent )
+exports.registerEvent = v1_oncall( [ 'memory_1GiB', 'long_timeout' ], registerEvent )
 exports.deleteEvent = v1_oncall( deleteEvent )
 
 // Email export to update event organisers
@@ -55,7 +55,7 @@ exports.getUniqueOrganiserEmails = v1_oncall( getUniqueOrganiserEmails )
 // QR Middleware API
 // ///////////////////////////////
 const claimMiddleware = require( './modules/claim' )
-exports.claimMiddleware = v2_onrequest( [ 'max_concurrency', 'keep_warm', 'memory' ], claimMiddleware )
+exports.claimMiddleware = v2_onrequest( [ 'max_concurrency', 'memory_2GiB' ], claimMiddleware )
 
 /* ///////////////////////////////
 // Kiosk generator middleware API
@@ -84,7 +84,7 @@ exports.updateEventAvailableCodes = functions.firestore.document( `codes/{codeId
 // Security
 // /////////////////////////////*/
 const { validateCallerDevice, validateCallerCaptcha } = require( './modules/security' )
-exports.validateCallerDevice = v2_oncall( [ 'high_memory', 'long_timeout', 'keep_warm' ], validateCallerDevice )
+exports.validateCallerDevice = v2_oncall( [ 'memory_512MiB', 'long_timeout' ], validateCallerDevice )
 exports.validateCallerCaptcha = v1_oncall( validateCallerCaptcha )
 
 // Log kiosk opens

functions/runtime/on_request_runtimes.js

@@ -12,6 +12,11 @@ exports.v1_onrequest = ( runtimes=[], handler ) => {
     // If the first parameter was a function, return the undecorated handler
     if( typeof runtimes === 'function' ) return functions.https.onRequest( runtimes )
 
+    // Check that all runtime keys exist in the v2_runtimes object
+    const runtime_keys = Object.keys( v1_runtimes )
+    const invalid_runtime_keys = runtimes.some( runtime_key => !runtime_keys.includes( runtime_key ) )
+    if( invalid_runtime_keys.length ) throw new Error( `Invalid runtime keys: ${ invalid_runtime_keys }` )
+
     // Config the runtimes for this function
     const runtime = runtimes.reduce( ( acc, runtime_key ) => ( { ...acc, ...v1_runtimes[ runtime_key ] } ), {} )
     return functions.runWith( runtime ).https.onRequest( handler )
@@ -32,6 +37,12 @@ exports.v2_onrequest = ( runtimes=[], handler ) => {
     // If the first parameter was a function, return the handler as 'protected' firebase oncall
     if( typeof runtimes === 'function' ) return onRequest( runtime_basis, runtimes )
 
+    // Check that all runtime keys exist in the v2_runtimes object
+    const runtime_keys = Object.keys( v2_runtimes )
+    const invalid_runtime_keys = runtimes.some( runtime_key => !runtime_keys.includes( runtime_key ) )
+    if( invalid_runtime_keys.length ) throw new Error( `Invalid runtime keys: ${ invalid_runtime_keys }` )
+
     const runtime = runtimes.reduce( ( acc, runtime_key ) => ( { ...acc, ...v2_runtimes[ runtime_key ] } ), runtime_basis )
+
     return onRequest( runtime, handler )
 }

functions/runtime/runtimes_settings.js

@@ -1,16 +1,22 @@
+// Function to reduce an array of memory declarations to an object of memory declarations
+const reduce_memory_declarations = ( acc, memory ) => ( { ...acc, [ `memory_${ memory }` ]: { memory } } )
+
 /**
+ * See https://firebase.google.com/docs/reference/functions/firebase-functions.runtimeoptions
  * @typedef {Object} V1runtimes
  * @property {string} high_memory - Allocate high memory to function
  * @property {string} long_timeout - Set long timeout to function
  * @property {string} keep_warm - Keep function warm
  */
 exports.v1_runtimes = {
-    high_memory: { memory: '4GB' },
+    // As per https://firebase.google.com/docs/reference/functions/firebase-functions.md#valid_memory_options
+    ...[ "128MB", "256MB", "512MB", "1GB", "2GB", "4GB", "8GB" ].reduce( reduce_memory_declarations, {} ),
     long_timeout: { timeoutSeconds: 540 },
     keep_warm: { minInstances: 1 },
 }
 
 /**
+ * See https://firebase.google.com/docs/reference/functions/2nd-gen/node/firebase-functions.globaloptions
  * @typedef {Object} V2runtimes
  * @property {string} protected - Enforce appcheck
  * @property {string} long_timeout - Set long timeout to function
@@ -21,7 +27,8 @@ exports.v1_runtimes = {
 exports.v2_runtimes = {
     protected: { enforceAppCheck: true },
     long_timeout: { timeoutSeconds: 540 },
-    high_memory: { memory: '4GiB' }, // Note: memory also increases compute power
+    // As per https://firebase.google.com/docs/reference/functions/2nd-gen/node/firebase-functions.md#memoryoption
+    ...[ "128MiB", "256MiB", "512MiB", "1GiB", "2GiB", "4GiB", "8GiB", "16GiB", "32GiB" ].reduce( reduce_memory_declarations, {} ),
     keep_warm: { minInstances: 1 },
     max_concurrency: { concurrency: 1000 }
 }