Merge pull request #38 from Project-OMOTES/change-result-error-handling

removed error handling from result handler

Author: mannershark

sdk/src/lib/handlers/ResultHandler.spec.ts

@@ -44,12 +44,18 @@ describe('ResultsHandler', () => {
 
     it('should error when not succeeded', (done) => {
       handler.getResult().subscribe({
-        next: () => {
-          done.fail('Should not have succeeded');
+        next: (result) => {
+        expect(result).toEqual({
+          uuid: 'uuid',
+          logs: 'logs',
+          outputEsdl: '',
+          resultType: JobResult.ResultType.ERROR,
+          esdlMessagesList: [],
+        });
+          done()
         },
-        error: (error) => {
-          expect(error.message).toEqual('Job uuid failed: logs');
-          done();
+        error: () => {
+          done.fail('should not have thrown error')
         },
         complete: () => {
           done.fail('Should not have completed');

sdk/src/lib/handlers/ResultHandler.ts

@@ -11,9 +11,6 @@ export class ResultHandler extends JobHandler {
     return messages$.pipe(
       map((message) => {
         const result = JobResult.deserializeBinary(message.content);
-        if (result.getResultType() === JobResult.ResultType.ERROR) {
-          throw new Error(`Job ${result.getUuid()} failed: ${result.getLogs()}`);
-        }
         return result.toObject();
       })
     );