refactor: update implementation to handle edge cases in eslint/rules/eol-open-bracket-spacing

PR-URL: #4254
Reviewed-by: Athan Reines <[email protected]>

Author: headlessNode

lib/node_modules/@stdlib/_tools/eslint/rules/eol-open-bracket-spacing/lib/main.js

@@ -85,15 +85,21 @@ function main( context ) {
 			node.arguments.length > 0
 		) {
 			args = node.arguments;
-			if ( args[ 0 ].type === 'ObjectExpression' ) {
+			if (
+				args[ 0 ].type === 'ObjectExpression' &&
+				args[ 0 ].properties.length > 0
+			) {
 				prevToken = source.getTokenBefore( args[ 0 ] );
 				tokenAfter = source.getFirstToken( args[ 0 ] );
 				if ( source.isSpaceBetween( prevToken, tokenAfter ) ) {
 					report( node, prevToken, tokenAfter );
 				}
 			} else if ( args[ 0 ].type === 'ArrayExpression' ) {
 				elem = args[ 0 ].elements[ 0 ];
-				if ( elem.type === 'ObjectExpression' ) {
+				if (
+					elem.type === 'ObjectExpression' &&
+					elem.properties.length > 0
+				) {
 					prevToken = source.getTokenBefore( args[ 0 ] );
 					tokenAfter = source.getFirstToken( args[ 0 ] );
 					if ( source.isSpaceBetween( prevToken, tokenAfter ) ) {
@@ -118,7 +124,10 @@ function main( context ) {
 			node.elements.length > 0
 		) {
 			elem = node.elements[ 0 ];
-			if ( elem.type === 'ObjectExpression' ) {
+			if (
+				elem.type === 'ObjectExpression' &&
+				elem.properties.length > 0
+			) {
 				prevToken = source.getFirstToken( node );
 				tokenAfter = source.getFirstToken( elem );
 

lib/node_modules/@stdlib/_tools/eslint/rules/eol-open-bracket-spacing/test/fixtures/unvalidated.js

@@ -35,6 +35,21 @@ test = {
 };
 valid.push( test);
 
+test = {
+	'code': [
+		'  var log = require( \'@stdlib/console/log\' );',
+		'  log( {} );'
+	].join( '\n' )
+};
+valid.push( test);
+
+test = {
+	'code': [
+		'var arr = [ {}, 1, 2, 3, 4 ];'
+	].join( '\n' )
+};
+valid.push( test);
+
 
 // EXPORTS //