Skip to content

Commit da6dfaa

Browse files
committed
Refactor: rename Mask -> Actions
1 parent a6fe6f8 commit da6dfaa

File tree

3 files changed

+18
-18
lines changed

3 files changed

+18
-18
lines changed

fanotify_api.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ type Event struct {
3535
// when NewListener is created by passing `true` with `withName` argument. The feature is available
3636
// only with kernels 5.9 or higher.
3737
FileName string
38-
// Mask holds bit mask representing the operation
39-
Mask Action
38+
// Actions holds bit mask representing the operations
39+
Actions Action
4040
// Pid Process ID of the process that caused the event
4141
Pid int
4242
}

fanotify_event.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -358,10 +358,10 @@ func (l *Listener) readEvents() error {
358358
mask = mask ^ unix.FAN_ONDIR
359359
}
360360
event := Event{
361-
Fd: int(metadata.Fd),
362-
Path: string(name[:n1]),
363-
Mask: Action(mask),
364-
Pid: int(metadata.Pid),
361+
Fd: int(metadata.Fd),
362+
Path: string(name[:n1]),
363+
Actions: Action(mask),
364+
Pid: int(metadata.Pid),
365365
}
366366
l.Events <- event
367367

@@ -407,7 +407,7 @@ func (l *Listener) readEvents() error {
407407
Fd: fd,
408408
Path: pathName,
409409
FileName: fileName,
410-
Mask: Action(mask),
410+
Actions: Action(mask),
411411
Pid: int(metadata.Pid),
412412
}
413413
l.Events <- event

fanotify_test.go

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ func TestWithCapSysAdmFanotifyFileAccessed(t *testing.T) {
9090
case event := <-l.Events:
9191
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
9292
assert.Equal(t, event.Pid, pid)
93-
hasFileAccessed := (event.Mask & FileAccessed) == FileAccessed
93+
hasFileAccessed := (event.Actions & FileAccessed) == FileAccessed
9494
assert.True(t, hasFileAccessed)
9595
}
9696
}
@@ -119,7 +119,7 @@ func TestWithCapSysAdmFanotifyFileModified(t *testing.T) {
119119
case event := <-l.Events:
120120
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
121121
assert.Equal(t, event.Pid, pid)
122-
isModifed := (event.Mask & FileModified) == FileModified
122+
isModifed := (event.Actions & FileModified) == FileModified
123123
assert.True(t, isModifed)
124124
}
125125
}
@@ -148,7 +148,7 @@ func TestWithCapSysAdmFanotifyFileClosed(t *testing.T) {
148148
case event := <-l.Events:
149149
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
150150
assert.Equal(t, event.Pid, pid)
151-
closeNoWrite := (event.Mask & FileClosedWithNoWrite) == FileClosedWithNoWrite
151+
closeNoWrite := (event.Actions & FileClosedWithNoWrite) == FileClosedWithNoWrite
152152
assert.True(t, closeNoWrite)
153153
}
154154
}
@@ -177,7 +177,7 @@ func TestWithCapSysAdmFanotifyFileOpen(t *testing.T) {
177177
case event := <-l.Events:
178178
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
179179
assert.Equal(t, event.Pid, pid)
180-
opened := (event.Mask & FileOpened) == FileOpened
180+
opened := (event.Actions & FileOpened) == FileOpened
181181
assert.True(t, opened)
182182
}
183183
}
@@ -202,7 +202,7 @@ func TestWithCapSysAdmFanotifyFileOrDirectoryOpen(t *testing.T) {
202202
case event := <-l.Events:
203203
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), fmt.Sprintf("%s/%s", watchDir, "."))
204204
assert.Equal(t, event.Pid, pid)
205-
opened := (event.Mask & FileOpened) == FileOpened
205+
opened := (event.Actions & FileOpened) == FileOpened
206206
assert.True(t, opened)
207207
}
208208
}
@@ -236,7 +236,7 @@ exit 0
236236
case event := <-l.Events:
237237
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
238238
assert.Equal(t, event.Pid, pid)
239-
openedForExec := (event.Mask & FileOpenedForExec) == FileOpenedForExec
239+
openedForExec := (event.Actions & FileOpenedForExec) == FileOpenedForExec
240240
assert.True(t, openedForExec)
241241
}
242242
}
@@ -270,7 +270,7 @@ exit 0
270270
case event := <-l.Events:
271271
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
272272
assert.Equal(t, event.Pid, pid)
273-
openedForExec := (event.Mask & FileAttribChanged) == FileAttribChanged
273+
openedForExec := (event.Actions & FileAttribChanged) == FileAttribChanged
274274
assert.True(t, openedForExec)
275275
}
276276
}
@@ -295,7 +295,7 @@ func TestWithCapSysAdmFanotifyFileCreated(t *testing.T) {
295295
case event := <-l.Events:
296296
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
297297
assert.Equal(t, event.Pid, pid)
298-
created := (event.Mask & FileCreated) == FileCreated
298+
created := (event.Actions & FileCreated) == FileCreated
299299
assert.True(t, created)
300300
}
301301
}
@@ -320,7 +320,7 @@ func TestWithCapSysAdmFanotifyFileOrDirectoryCreated(t *testing.T) {
320320
case event := <-l.Events:
321321
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testDir)
322322
assert.Equal(t, event.Pid, pid)
323-
created := (event.Mask & FileCreated) == FileCreated
323+
created := (event.Actions & FileCreated) == FileCreated
324324
assert.True(t, created)
325325
}
326326
}
@@ -350,7 +350,7 @@ func TestWithCapSysAdmFanotifyFileDeleted(t *testing.T) {
350350
case event := <-l.Events:
351351
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testFile)
352352
assert.Equal(t, event.Pid, pid)
353-
deleted := (event.Mask & FileDeleted) == FileDeleted
353+
deleted := (event.Actions & FileDeleted) == FileDeleted
354354
assert.True(t, deleted)
355355
}
356356
}
@@ -380,7 +380,7 @@ func TestWithCapSysAdmFanotifyFileOrDirDeleted(t *testing.T) {
380380
case event := <-l.Events:
381381
assert.Equal(t, fmt.Sprintf("%s/%s", event.Path, event.FileName), testDir)
382382
assert.Equal(t, event.Pid, pid)
383-
deleted := (event.Mask & FileDeleted) == FileDeleted
383+
deleted := (event.Actions & FileDeleted) == FileDeleted
384384
assert.True(t, deleted)
385385
}
386386
}

0 commit comments

Comments
 (0)