added quick testcases for SuperpixelSLIC and SuperpixelLSC + added SuperpixelSLIC algorithm constants

Author: justadudewhohacks

cc/modules/ximgproc/SuperpixelLSC.cc

@@ -35,7 +35,7 @@ NAN_METHOD(SuperpixelLSC::New) {
 	FF_ARG_INSTANCE(0, self->img, Mat::constructor, FF_UNWRAP_MAT_AND_GET);
   FF_ARG_INT_IFDEF(1, self->regionSize, 10);
   FF_ARG_INT_IFDEF(2, self->ratio, 0.075);
-	
+
 	self->Wrap(info.Holder());
 	self->superpixelLsc = cv::ximgproc::createSuperpixelLSC(
 		self->img,

cc/modules/ximgproc/SuperpixelSLIC.cc

@@ -34,7 +34,7 @@ NAN_METHOD(SuperpixelSLIC::New) {
 	SuperpixelSLIC* self = new SuperpixelSLIC();
 
 	FF_ARG_INSTANCE(0, self->img, Mat::constructor, FF_UNWRAP_MAT_AND_GET);
-	FF_ARG_INT(1, self->algorithm);
+	FF_ARG_INT_IFDEF(1, self->algorithm, cv::ximgproc::SLICO);
   FF_ARG_INT_IFDEF(2, self->regionSize, 10);
   FF_ARG_INT_IFDEF(3, self->ruler, 10.0);
 

cc/modules/ximgproc/ximgproc.cc

@@ -13,6 +13,8 @@ NAN_MODULE_INIT(XImgproc::Init) {
 #if CV_MINOR_VERSION > 0
 	SuperpixelSLIC::Init(target);
 	SuperpixelLSC::Init(target);
+	FF_SET_JS_PROP(target, SLIC, Nan::New<v8::Integer>(cv::ximgproc::SLIC));
+	FF_SET_JS_PROP(target, SLICO, Nan::New<v8::Integer>(cv::ximgproc::SLICO));
 #endif
 }
 

doc/ximgproc/SuperpixelSLIC.md

@@ -17,7 +17,7 @@
 
 ## Constructors
 ``` javascript
-SuperpixelSLIC : new SuperpixelSLIC(Mat img, Int algorithm, Int regionSize = 10, Number ruler = 10)
+SuperpixelSLIC : new SuperpixelSLIC(Mat img, Int algorithm = cv.SLICO, Int regionSize = 10, Number ruler = 10)
 ```
 
 ## Methods

test/tests/modules/imgproc/ximgproc.test.js

@@ -2,7 +2,7 @@ const cv = global.dut;
 const { assertMetaData, assertPropsWithValue, funcShouldRequireArgs, readTestImage } = global.utils;
 const { assert, expect } = require('chai');
 
-describe('ximgproc', () => {
+describe.only('ximgproc', () => {
   if (!cv.xmodules.ximgproc) {
     it('compiled without ximgproc');
     return;
@@ -43,4 +43,58 @@ describe('ximgproc', () => {
       });
     });
   });
+
+  if (cv.version.minor > 0) {
+    describe('SuperpixelSLIC', () => {
+      const algorithm = cv.SLICO;
+
+      describe('constructor', () => {
+        funcShouldRequireArgs(() => new cv.SuperpixelSLIC());
+
+        it('should be constructable with required args', () => {
+          const superpixel = new cv.SuperpixelSLIC(testImg, algorithm);
+          expect(superpixel).to.have.property('img').instanceOf(cv.Mat);
+          assertMetaData(testImg)(superpixel.img);
+          assertPropsWithValue(superpixel)({
+            algorithm
+          });
+        });
+      });
+
+      describe('iterate', () => {
+        it('should iterate with default values', () => {
+          const superpixel = new cv.SuperpixelSLIC(testImg, algorithm);
+          superpixel.iterate();
+          assert(superpixel.numCalculatedSuperpixels > 0, 'no superpixels calculated');
+          assertPropsWithValue(superpixel.labels)({
+            rows: testImg.rows, cols: testImg.cols, type: cv.CV_32S
+          });
+        });
+      });
+    });
+
+
+    describe('SuperpixelLSC', () => {
+      describe('constructor', () => {
+        funcShouldRequireArgs(() => new cv.SuperpixelLSC());
+
+        it('should be constructable with required args', () => {
+          const superpixel = new cv.SuperpixelLSC(testImg);
+          expect(superpixel).to.have.property('img').instanceOf(cv.Mat);
+          assertMetaData(testImg)(superpixel.img);
+        });
+      });
+
+      describe('iterate', () => {
+        it('should iterate with default values', () => {
+          const superpixel = new cv.SuperpixelLSC(testImg);
+          superpixel.iterate();
+          assert(superpixel.numCalculatedSuperpixels > 0, 'no superpixels calculated');
+          assertPropsWithValue(superpixel.labels)({
+            rows: testImg.rows, cols: testImg.cols, type: cv.CV_32S
+          });
+        });
+      });
+    });
+  }
 });