[eslint] duplicate conditions in indices and nodes API

[ananzh]

What is the bug?
eslint shows two APIs have duplicate conditions.

yarn lint
yarn run v1.22.5
$ eslint .

/opensearch-js/api/api/indices.js
  1112:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if
  1115:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if
  1118:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if
  1127:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if

/opensearch-js/api/api/nodes.js
  169:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if
  180:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if
  183:14  error  This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain  no-dupe-else-if

✖ 7 problems (7 errors, 0 warnings)

How can one reproduce the bug?
After eslint PR is merged

1. Go to '.eslintrc.json'
2. Remove ' "no-dupe-else-if": "off"'
3. Run yarn lint
4. See error

What is the expected behavior?
Research on this
Remove duplicate conditions

[ananzh]

duplication:

opensearch-js/api/api/nodes.js

Lines 116 to 131 in 5328c31

	if ((node_id || nodeId) != null) {
	if (method == null) method = 'GET'
	path = '/' + '_nodes' + '/' + encodeURIComponent(node_id || nodeId) + '/' + 'hot_threads'
	} else if ((node_id || nodeId) != null) {
	if (method == null) method = 'GET'
	path = '/' + '_cluster' + '/' + 'nodes' + '/' + encodeURIComponent(node_id || nodeId) + '/' + 'hotthreads'
	} else if ((node_id || nodeId) != null) {
	if (method == null) method = 'GET'
	path = '/' + '_nodes' + '/' + encodeURIComponent(node_id || nodeId) + '/' + 'hotthreads'
	} else if ((node_id || nodeId) != null) {
	if (method == null) method = 'GET'
	path = '/' + '_cluster' + '/' + 'nodes' + '/' + encodeURIComponent(node_id || nodeId) + '/' + 'hot_threads'
	} else {
	if (method == null) method = 'GET'
	path = '/' + '_nodes' + '/' + 'hot_threads'
	}

[frost017]

Good day @ananzh !
I want to let you know that I am working on this issue. Thank you for this opportunity!

[nhtruong]

Closing since this has been replaced by generated API functions.