Gurke Js Rückruf Problem? oder Feature-Problem?

Question

würde Ich mag eine Funktion wie diese schreiben:

Scenario: new Singleton create 
    When a new, unmatchable identity is received 
    Then a new tin record should be created 
    And a new bronze record should be created 
    And a new gold record should be created 

, die Schritte wie diese binden würde:

defineSupportCode(function ({ Before, Given, Then, When }) { 

    var expect = require('chai').expect; 
    var chanceGenerator = require('./helpers/chanceGenerator') 
    var request = require('./helpers/requestGenerator') 

    let identMap; 

    // reset identMap before each scenario 
    Before(function() { 
    identMap = []; 
    }); 

    // should generate a valid identity 
    // persist it in a local variable so it can be tested in later steps 
    // and persist to the db via public endpoint 
    When('a new, unmatchable identity is received', function (callback) { 
    identMap.push(chanceGenerator.identity()); 
    request.pubPostIdentity(identMap[identMap.length-1], callback); 
    }); 

    // use the local variable to retrieve Tin that was persisted 
    // validate the tin persisted all the props that it should have 
    Then('a new tin record should be created', function (callback) { 
    request.pubGetIdentity(identMap[identMap.length-1], callback); 

    // var self = this; 

    // request.pubGetIdentity(identMap[identMap.length-1], callback,() => { 
    // console.log('never gets here...'); 
    // self.callback(); 
    // callback(); 
    // }); 

    // request.pubGetIdentity(identMap[identMap.length-1], (callback) => { 
    // console.log('never gets here...'); 
    // self.callback(); 
    // callback(); 
    // }); 

    }); 

Das Problem, das ich habe ist, dass ich tun kann, nichts im Then Callback. An dieser Stelle möchte ich überprüfen können, ob die Antwort die richtigen Daten enthält.

Hier sind relevante Auszüge aus den Hilfsdateien:

var pubPostIdentity = function (ident, callback) { 
    console.log('pubIdentity'); 
    var options = { 
    method: 'POST', 
    url: 'http://cucumber.utu.ai:4020/identity/' + ident.platform + '/' + ident.platformId, 
    headers: { 
     'X-Consumer-Custom-Id': ident.botId + '_' + ident.botId 
    }, 
    body: JSON.stringify(ident) 
    }; 
    console.log('ident: ', ident); 
    request(options, (err, response, body) => { 
    if (err) { 
     console.log('pubPostIdentity: ', err); 
     callback(err); 
    } 
    console.log('pubPostIdentity: ', response.statusCode); 
    callback(); 
    }); 
} 

// accept an identity and retrieve from staging via identity public endpoint 
var pubGetIdentity = function (ident, callback) { 
    console.log('pubGetIdentity'); 
    var options = { 
    method: 'GET', 
    url: 'http://cucumber.utu.ai:4020/identity/' + ident.platform + '/' + ident.platformId, 
    headers: { 
     'X-Consumer-Custom-Id': ident.botId + '_' + ident.botId 
    } 
    }; 
    request(options, (err, response) => { 
    if (err) { 
     console.log('pubGetIdentity: ', err); 
     callback(err); 
    } 
    console.log('pubGetIdentity: ', response.body); 
    callback(); 
    }); 
} 

Etwas, das wir betrachten, als eine Option ist die Funktion neu zu schreiben, eine andere Schrittdefinition Struktur zu passen. Wenn wir wieder schrieb die Funktion wie folgt aus:

Scenario: new Singleton create 
    When a new, unmatchable 'TIN_RECORD' is received 
    Then the Identity Record should be created successfully 
    When the Identity Record is retreived for 'tin' 
    Then a new 'tin' should be created 
    When the Identity Record is retreived for 'bronze' 
    Then a new 'bronze' should be created 
    When the Identity Record is retreived for 'gold' 
    Then a new 'gold' should be created 

Ich glaube, es den Rist Rückruf Problem umgeht wir ringen mit, aber ich hasse es, den Zusammenbruch der Funktion. Dadurch wird das Feature für das Unternehmen weniger lesbar und verständlich.

Also ... meine Frage, die Zusammenfassung zuerst präsentiert, ist es falsch geschrieben? Versuche ich, Schrittdefinitionen zu bekommen, um etwas zu tun, was sie nicht tun sollten? Oder ist mein Mangel an Js Fähigkeiten hell, und das sollte sehr machbar sein, ich mische nur die Rückrufe?

Answer 1

Erstens würde ich sagen, dass Ihre neu geschriebene Funktion falsch ist. Sie sollten nie in der Reihenfolge Gegeben, Wann, Dann zurückgehen. Du gehst vom Dann zum Wann zurück, was falsch ist.

Given wird zum Einrichten von Vorbedingungen verwendet. Wann wird für den eigentlichen Test verwendet? Dann wird für die Behauptungen verwendet. Jedes Szenario sollte ein einzelner Test sein und sollte daher sehr wenige When-Klauseln enthalten. Wenn Sie möchten, können Sie mit Szenario-Outlines mehrere sehr ähnliche Tests miteinander kombinieren.

In diesem Fall wird empfohlen, es zurück zu den ersten Prinzipien zu nehmen und zu sehen, ob das funktioniert. Dann baue dich langsam auf, um loszulegen.

Ich vermute in diesem Fall, dass das Problem in einer Ausnahme geworfen wird, die nicht behandelt wird. Sie könnten versuchen, es neu zu schreiben, um stattdessen Versprechen zu verwenden, die dann bei einem Fehler zurückgewiesen werden. Das gibt eine bessere Fehlerberichterstattung.