Skip to content

Commit f469d19

Browse files
committed
[all] Fix lint warnings
Signed-off-by: Oleksandr Grytsov <oleksandr_grytsov@epam.com>
1 parent d59530b commit f469d19

File tree

5 files changed

+18
-17
lines changed

5 files changed

+18
-17
lines changed

database/database_internal_test.go

+6-5
Original file line numberDiff line numberDiff line change
@@ -401,7 +401,7 @@ func TestMultiThread(t *testing.T) {
401401
go func() {
402402
defer wg.Done()
403403

404-
for i := 0; i < numIterations; i++ {
404+
for i := range numIterations {
405405
if err := db.SetOperationVersion(uint64(i)); err != nil {
406406
t.Errorf("Can't set operation version: %s", err)
407407

@@ -424,7 +424,7 @@ func TestMultiThread(t *testing.T) {
424424
go func() {
425425
defer wg.Done()
426426

427-
for i := 0; i < numIterations; i++ {
427+
for i := range numIterations {
428428
if err := db.SetJournalCursor(strconv.Itoa(i)); err != nil {
429429
t.Errorf("Can't set journal cursor: %s", err)
430430

@@ -436,7 +436,7 @@ func TestMultiThread(t *testing.T) {
436436
go func() {
437437
defer wg.Done()
438438

439-
for i := 0; i < numIterations; i++ {
439+
for range numIterations {
440440
if _, err := db.GetJournalCursor(); err != nil {
441441
t.Errorf("Can't get journal cursor: %s", err)
442442

@@ -578,11 +578,12 @@ func TestInstances(t *testing.T) {
578578
const (
579579
testServiceID = "testService"
580580
testSubjectID = "testSubject"
581+
numInstances = 5
581582
)
582583

583-
var allInstances []launcher.InstanceInfo
584+
allInstances := make([]launcher.InstanceInfo, 0, numInstances)
584585

585-
for i := 0; i < 5; i++ {
586+
for i := range numInstances {
586587
subjectInstance := launcher.InstanceInfo{
587588
InstanceInfo: aostypes.InstanceInfo{
588589
InstanceIdent: aostypes.InstanceIdent{

launcher/launcher_test.go

+5-5
Original file line numberDiff line numberDiff line change
@@ -3024,8 +3024,8 @@ func compareArrays(len1, len2 int, isEqual func(index1, index2 int) bool) bool {
30243024
}
30253025

30263026
loop1:
3027-
for i := 0; i < len1; i++ {
3028-
for j := 0; j < len2; j++ {
3027+
for i := range len1 {
3028+
for j := range len2 {
30293029
if isEqual(i, j) {
30303030
continue loop1
30313031
}
@@ -3036,8 +3036,8 @@ loop1:
30363036

30373037
loop2:
30383038

3039-
for j := 0; j < len2; j++ {
3040-
for i := 0; i < len1; i++ {
3039+
for j := range len2 {
3040+
for i := range len1 {
30413041
if isEqual(i, j) {
30423042
continue loop2
30433043
}
@@ -3342,7 +3342,7 @@ func getSystemGroups(n int) (groups []string, err error) {
33423342
systemGroups = append(systemGroups, group)
33433343
}
33443344

3345-
for i := 0; i < n; i++ {
3345+
for i := range n {
33463346
groups = append(groups, systemGroups[i%len(systemGroups)])
33473347
}
33483348

logging/logging_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ func TestGetSystemLog(t *testing.T) {
164164
till = from.Add(5 * time.Second)
165165
)
166166

167-
for i := 0; i < 20; i++ {
167+
for range 20 {
168168
testJournal.addMessage("Hello World", "logger", "", "2")
169169
}
170170

@@ -307,7 +307,7 @@ func TestMaxPartCountLog(t *testing.T) {
307307
till = from.Add(20 * time.Second)
308308
)
309309

310-
for i := 0; i < 200; i++ {
310+
for i := range 200 {
311311
testJournal.addMessage(fmt.Sprintf("Super mega log %d", i),
312312
unitName, "/system.slice/system-aos@service.slice/"+unitName, "2")
313313
}

networkmanager/networkmanager.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -588,7 +588,7 @@ func (manager *NetworkManager) createNetwork(
588588

589589
func (manager *NetworkManager) generateVlanName() (string, error) {
590590
next:
591-
for i := 0; i < countRetryVlanNameGeneration; i++ {
591+
for range countRetryVlanNameGeneration {
592592
vlanName, err := randomVlanName()
593593
if err != nil {
594594
return "", err

networkmanager/networkmanager_test.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -497,15 +497,15 @@ func TestUpdateNetwork(t *testing.T) {
497497
t.Fatalf("Can't update networks: %v", err)
498498
}
499499

500-
for i := 0; i < 2; i++ {
500+
for range 2 {
501501
select {
502502
case <-vlanCreator.createVlanCh:
503503
case <-time.After(time.Second):
504504
t.Fatalf("Can't create vlan")
505505
}
506506
}
507507

508-
for i := 0; i < 2; i++ {
508+
for range 2 {
509509
select {
510510
case <-storage.chanAddNetwork:
511511
case <-time.After(time.Second):
@@ -934,7 +934,7 @@ func TestTrafficMonitoring(t *testing.T) {
934934
t.Error("Unexpected system traffic")
935935
}
936936

937-
for i := 0; i < 3; i++ {
937+
for i := range 3 {
938938
iptableInterface.waitUpdateIptablesCache()
939939

940940
in, out, err = manager.GetInstanceTraffic("instance1")
@@ -1440,7 +1440,7 @@ func (iptables *testIPTablesInterface) ListAllRulesWithCounters(table string) ([
14401440
var counters []string
14411441

14421442
for chain, iptablesData := range iptables.chain {
1443-
for i := 0; i < iptablesData.countChain; i++ {
1443+
for range iptablesData.countChain {
14441444
counters = append(counters, fmt.Sprintf("%s -c 0 %d", chain, iptablesData.limit))
14451445
}
14461446

0 commit comments

Comments
 (0)