Fix the remaining tests

Author: Dexterp37

glean/tests/core/glean.spec.ts

@@ -441,7 +441,7 @@ describe("Glean", function() {
     //
     // This disables the uploading and deletion of pings from the pings database,
     // this allows us to query the database to check that our pings are as expected.
-    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve());
+    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve([]));
 
     const custom = new PingType({
       name: "custom",

glean/tests/core/metrics/labeled.spec.ts

@@ -22,7 +22,7 @@ describe("LabeledMetric", function() {
   beforeEach(async function() {
     await Glean.testResetGlean(testAppId);
     // Disable ping uploading for it not to interfere with this tests.
-    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve());
+    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve([]));
   });
 
   afterEach(function () {

glean/tests/core/pings/index.spec.ts

@@ -36,7 +36,7 @@ describe("PingType", function() {
 
   it("collects and stores ping on submit", async function () {
     // Disable ping uploading for it not to interfere with this tests.
-    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve());
+    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve([]));
 
     const ping = new PingType({
       name: "custom",
@@ -59,7 +59,7 @@ describe("PingType", function() {
 
   it("empty pings with send if empty flag are submitted", async function () {
     // Disable ping uploading for it not to interfere with this tests.
-    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve());
+    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve([]));
 
     const ping1 = new PingType({
       name: "ping1",

glean/tests/core/pings/maker.spec.ts

@@ -141,7 +141,7 @@ describe("PingMaker", function() {
 
   it("collect and store triggers the AfterPingCollection and deals with possible result correctly", async function () {
     // Disable ping uploading for it not to interfere with this tests.
-    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve());
+    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve([]));
 
     await Glean.testResetGlean(testAppId, true, { plugins: [ new MockPlugin() ]});
     const ping = new PingType({
@@ -162,7 +162,7 @@ describe("PingMaker", function() {
 
   it("ping payload is logged before it is modified by a plugin", async function () {
     // Disable ping uploading for it not to interfere with this tests.
-    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve());
+    sandbox.stub(Glean["pingUploader"], "triggerUpload").callsFake(() => Promise.resolve([]));
 
     await Glean.testResetGlean(testAppId, true, {
       debug: {