Skip to content

Commit a26430b

Browse files
refactor(dapp): rename dataset to protected-data in sendEmail results
1 parent 38ec12b commit a26430b

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

dapp/src/sendEmail.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -214,11 +214,9 @@ async function start() {
214214
'total-processed': results.length,
215215
'success-count': successCount,
216216
'error-count': errorCount,
217-
'dataset-results': results.map((r) => ({
217+
'protected-data-results': results.map((r) => ({
218218
index: r.index,
219-
dataset:
220-
process.env[`IEXEC_DATASET_${r.index}_FILENAME`] ||
221-
`dataset-${r.index}`,
219+
protectedData: process.env[`IEXEC_DATASET_${r.index}_FILENAME`],
222220
response: r.response,
223221
})),
224222
};
@@ -229,7 +227,7 @@ async function start() {
229227
);
230228
}
231229

232-
const computedData = {
230+
const computedData = {
233231
'deterministic-output-path': `${workerEnv.IEXEC_OUT}/result.txt`,
234232
};
235233

dapp/tests/e2e/app.test.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -487,8 +487,8 @@ describe('sendEmail', () => {
487487
expect(result).toHaveProperty('total-processed', 2);
488488
expect(result).toHaveProperty('success-count');
489489
expect(result).toHaveProperty('error-count');
490-
expect(result).toHaveProperty('dataset-results');
491-
expect(result['dataset-results']).toHaveLength(2);
490+
expect(result).toHaveProperty('protected-data-results');
491+
expect(result['protected-data-results']).toHaveLength(2);
492492
});
493493

494494
it('should handle mixed valid and invalid datasets', async () => {
@@ -506,13 +506,13 @@ describe('sendEmail', () => {
506506
const result = JSON.parse(resultTxt);
507507

508508
expect(result).toHaveProperty('total-processed', 2);
509-
expect(result['dataset-results']).toHaveLength(2);
509+
expect(result['protected-data-results']).toHaveLength(2);
510510

511511
// Should have both successes and errors
512-
const successResults = result['dataset-results'].filter(
512+
const successResults = result['protected-data-results'].filter(
513513
(r) => r.response.status === 200
514514
);
515-
const errorResults = result['dataset-results'].filter(
515+
const errorResults = result['protected-data-results'].filter(
516516
(r) => r.response.status !== 200
517517
);
518518

@@ -609,11 +609,11 @@ describe('sendEmail', () => {
609609
expect(result).toHaveProperty('total-processed', 2);
610610
expect(result).toHaveProperty('success-count', 2);
611611
expect(result).toHaveProperty('error-count', 0);
612-
expect(result).toHaveProperty('dataset-results');
613-
expect(result['dataset-results']).toHaveLength(2);
612+
expect(result).toHaveProperty('protected-data-results');
613+
expect(result['protected-data-results']).toHaveLength(2);
614614

615615
// Verify each dataset result
616-
result['dataset-results'].forEach((datasetResult, index) => {
616+
result['protected-data-results'].forEach((datasetResult, index) => {
617617
expect(datasetResult).toHaveProperty('index', index + 1);
618618
expect(datasetResult).toHaveProperty('dataset');
619619
expect(datasetResult).toHaveProperty('response');
@@ -660,7 +660,7 @@ describe('sendEmail', () => {
660660
);
661661
expect(result).toHaveProperty('status', 200);
662662
expect(result).not.toHaveProperty('total-processed');
663-
expect(result).not.toHaveProperty('dataset-results');
663+
expect(result).not.toHaveProperty('protected-data-results');
664664

665665
// Verify computed.json structure
666666
expect(computed).toHaveProperty('deterministic-output-path');

0 commit comments

Comments
 (0)