fix: component lifecycle methods should always return Promise

packages/atlas/test/aliasing.test.mjs

@@ -43,7 +43,7 @@ describe('Atlas: cross-component communication', () => {
     }
 
     class DummyService extends Service {
-      prepare() { return api }
+      prepare() { return Promise.resolve(api) }
     }
 
     class DummyAction extends Action {
@@ -92,7 +92,7 @@ describe('Atlas: cross-component communication', () => {
     class InternalService extends Service {
       static internal = true
 
-      prepare() { return api }
+      prepare() { return Promise.resolve(api) }
     }
 
     class DummyAction extends Action {

packages/aws/src/service.mjs

@@ -37,7 +37,7 @@ class AWS extends Service {
       aws[service] = new AWSService(config)
     }
 
-    return aws
+    return Promise.resolve(aws)
   }
 
   stop() {

packages/mongoose/src/service.mjs

@@ -34,7 +34,7 @@ class Mongoose extends Service {
     // Add a trace logger to allow users to monitor Mongoose activity
     instance.set('debug', this::mtrace)
 
-    return instance
+    return Promise.resolve(instance)
   }
 
   async start(instance) {

packages/nodemailer/src/service.mjs

@@ -52,7 +52,7 @@ class Nodemailer extends Service {
     // Attach a promisified version of the sendMail function
     instance.send = instance::send
 
-    return instance
+    return Promise.resolve(instance)
   }
 
   async stop(instance) {

packages/sequelize/src/service.mjs

@@ -21,7 +21,7 @@ class Sequelize extends Service {
     // Add a trace logger to allow users to monitor Sequelize activity
     this.config.options.logging = this::strace
 
-    return new Database(this.config.uri, this.config.options)
+    return Promise.resolve(new Database(this.config.uri, this.config.options))
   }
 
   async start(instance) {