Moving the model file deletion into the make.js script

Summary: This ensures that Arc-diff jobs will actually work, and removes the dependency on the bat files, which are only supposed to be a utility helper for customers.

Test Plan: Jenker!

Reviewers: Marco.Williams

Subscribers: jenkins, #sdkdiff-actionscriptsdk

Differential Revision: https://phab.playfabdev.com/D3012

Author: pgilmorepf

SDKBuildScripts/actionscript_build.bat

@@ -1,19 +1,3 @@
-rem === Cleaning existing files from ActionScriptSDK ===
-pushd ..\..\sdks\ActionScriptSDK\PfApiTest\com\playfab
-pushd AdminModels
-del *.as >nul 2>&1
-popd
-pushd ClientModels
-del *.as >nul 2>&1
-popd
-pushd MatchmakerModels
-del *.as >nul 2>&1
-popd
-pushd ServerModels
-del *.as >nul 2>&1
-popd
-popd
-
 pushd ..
 if [%1] == [] (
 rem === BUILDING ActionScriptSDK ===

targets/actionscript/make.js

@@ -1,12 +1,15 @@
-var path = require("path");
+var fs = require("fs");
 var ejs = require("ejs");
+var path = require("path");
 
 exports.putInRoot = true;
 
 exports.makeCombinedAPI = function (apis, sourceDir, apiOutputDir) {
-    console.log("Generating ActionScript3 combined SDK to " + apiOutputDir);
-    
     apiOutputDir = path.resolve(apiOutputDir, "PfApiTest"); // This is an oddity in the ActionScriptSDK which we shouldn't resolve until we do a major revision number change
+
+    console.log("Generating ActionScript3 combined SDK to " + apiOutputDir);
+
+    RemoveExcessFiles(apiOutputDir);
     copyTree(path.resolve(sourceDir, "source"), apiOutputDir);
 
     for (var i = 0; i < apis.length; i++) {
@@ -17,6 +20,25 @@ exports.makeCombinedAPI = function (apis, sourceDir, apiOutputDir) {
     GenerateSimpleFiles(apis, sourceDir, apiOutputDir);
 }
 
+function RemoveFilesInDir(dirPath, searchFilter) {
+    var files;
+    try { files = fs.readdirSync(dirPath); }
+      catch (e) { return; }
+    if (files.length > 0)
+        for (var i = 0; i < files.length; i++) {
+            var filePath = path.resolve(dirPath, files[i]);
+            if (fs.statSync(filePath).isFile() && (!searchFilter || filePath.contains(searchFilter)))
+                fs.unlinkSync(filePath);
+        }
+};
+
+function RemoveExcessFiles(apiOutputDir) {
+    RemoveFilesInDir(path.resolve(apiOutputDir, "com/playfab/AdminModels"), ".as");
+    RemoveFilesInDir(path.resolve(apiOutputDir, "com/playfab/ClientModels"), ".as");
+    RemoveFilesInDir(path.resolve(apiOutputDir, "com/playfab/MatchmakerModels"), ".as");
+    RemoveFilesInDir(path.resolve(apiOutputDir, "com/playfab/ServerModels"), ".as");
+}
+
 function GetBaseTypeSyntax(datatype) {
     // The model-inheritance feature was removed.
     // However in the future, we may still use some inheritance links for request/result baseclasses, for other sdk features